Merged
Conversation
… the event to their user page and to their profile in the DB. For non-users it takes them to the login/signup page. This change required a fair bit of refactoring of a number of different parts of code. The events / users api's now use axios. The eventsPage now contains more logic for the save and displaying of user events and the subsequent components pass more of this state/props through. Fixed navbar bug by adding a defined bg color. Use events commented out for the time being. Filter now set to all on load rather than just music.
…ontend into saveUserEvents
… numbers remain numbers. Forms auto to strings.
arecouz
approved these changes
Jun 14, 2025
Contributor
arecouz
left a comment
arecouz
left a comment
There was a problem hiding this comment.
Looks good! We should get rid of the console.logs though. Can have a clean up on Monday
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
✨ What’s Changed?
Added heart save button and populate userEvents with their saved events with working filters. Theres a fair bit going on so just have a read through and let me know if you've got any questions.
Some slight styling changes to the event and made the url property a link so it takes them to the website.
Slight fix to edit events. The form was making all elements string but price and distance are supposed to be numbers on the backend. Added a parseValue function/check and all working well.
Changed the get event api to use axios. Created saveUserevents api's using axios
📚 Related Issues
✅ Checklist
npm run lint / run lint:fix)npm run type-check)