Fixed word cloud data issues #159

Merged
snedadah merged 4 commits from shahanneda/fix-word-cloud-issues into main 2023-02-05 13:40:09 -05:00
Owner
No description provided.
snedadah added 1 commit 2023-01-31 00:51:08 -05:00
continuous-integration/drone/push Build is passing Details
7895765250
Fix word cloud data issues
snedadah requested review from e26chiu 2023-01-31 00:51:17 -05:00
snedadah reviewed 2023-01-31 00:52:18 -05:00
@ -169,3 +169,3 @@
<ComponentWrapper
heading="What is your favourite restaurant to go to on/around campus?"
bodyText="With all the diverse restaurant around campus, it seems like the most popular choices of food are sushi, noodles, and shawarma (Lazeez hehehe)."
bodyText="With all the diverse restaurant around campus, it seems like the most popular choices of food are sushi, noodles, and shawarma."
Author
Owner

oops, idk why i did this. will be override later anyway when we copy all the text.

oops, idk why i did this. will be override later anyway when we copy all the text.
snedadah added 1 commit 2023-01-31 00:53:33 -05:00
snedadah added 1 commit 2023-01-31 00:54:05 -05:00
continuous-integration/drone/push Build is passing Details
6ab6b47843
Fixed data
e26chiu added 1 commit 2023-02-04 13:20:56 -05:00
continuous-integration/drone/push Build is passing Details
301a1f76f0
Fix grammar mistake
e26chiu approved these changes 2023-02-04 13:22:19 -05:00
e26chiu left a comment
Contributor

LGTM! I have a few non-blocking questions about miscellaneous data.

LGTM! I have a few non-blocking questions about miscellaneous data.
@ -218,7 +218,7 @@ export const M7 = [
value: 3,
Contributor

Some of these categories are repeat of each other. E.g. Communication, Become more social, (to a lesser extent) Improved relationship
E.g. Responsability, Grown as a person, Mature, Life skills

Some of these categories are repeat of each other. E.g. Communication, Become more social, (to a lesser extent) Improved relationship E.g. Responsability, Grown as a person, Mature, Life skills
Author
Owner

I don't think we should manually put these together? I feel like that would be too much interfering, its interesting to see different thoughts on the same thing imo.

I don't think we should manually put these together? I feel like that would be too much interfering, its interesting to see different thoughts on the same thing imo.
@ -319,3 +319,3 @@
},
{
text: "Mindfullness & Mature",
text: "Mindfulness & Mature",
Contributor

Mindfulness & Mature sounds weird, I don't know if it's just me... I prefer keeping mindfulness only.

Mindfulness & Mature sounds weird, I don't know if it's just me... I prefer keeping mindfulness only.
Author
Owner

can't seem to find it anymore?

can't seem to find it anymore?
@ -323,3 +323,3 @@
},
{
text: "Self regulation",
text: "Self-regulation",
Contributor

Is "Have sex" comment SFW?

Is "Have sex" comment SFW?
Author
Owner

I think its fine.

I think its fine.
snedadah merged commit 8c0b5f8556 into main 2023-02-05 13:40:09 -05:00
snedadah deleted branch shahanneda/fix-word-cloud-issues 2023-02-05 13:40:10 -05:00
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: www/cs-2022-class-profile#159
No description provided.