Skip to content

Reactions Event Card#23

Open
gavinlin24 wants to merge 7 commits intomainfrom
feature/gavinlin24-mk7652/FE-ReactionEventCard
Open

Reactions Event Card#23
gavinlin24 wants to merge 7 commits intomainfrom
feature/gavinlin24-mk7652/FE-ReactionEventCard

Conversation

@gavinlin24
Copy link
Collaborator

Created an event card with thumb- up/down buttons. Also added a new collection to the database for storing event reaction data.

gavinlin24 and others added 7 commits January 8, 2025 16:50
Created new event card with like/dislike button
Added GET/POST reaction routes
Changed userId property to userEmail
Added userEmail and eventId props to ThumbButtons
Connected ThumbButtons with backend
Copy link
Collaborator

@nikhilitis nikhilitis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great for the most part! The code is readable, functions correctly on the website, and is easy to follow.

Just one comment:
When I run the site on your branch, I get the following output on stdout for the backend:

ValidationError: Reactions validation failed: reactionType: Path `reactionType` is required.
    at model.Document.invalidate (/Users/nikhilrao/cses/cses_webdev/backend/node_modules/mongoose/lib/document.js:3157:32)
    at /Users/nikhilrao/cses/cses_webdev/backend/node_modules/mongoose/lib/document.js:2950:17
    at /Users/nikhilrao/cses/cses_webdev/backend/node_modules/mongoose/lib/schematype.js:1368:9
    at processTicksAndRejections (node:internal/process/task_queues:78:11)

I'm not sure what specific action is triggering this, but it would be nice to have it fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants