Implement per-category page #86
Merged
Conversation
This file contains 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
-axios serializes arrays differently from how fastapi parses arrays in a URL param -Implement a custom serializer on the frontend for the category_ids param used for the /events backend -This fixes the home page top events component: now it displays user-specific categories instead of top events from any category
-Rename 'Science & Tech' to 'Science & Technology' in the frontend Category enum to match the backend naming -This fixes the display of the 'Science & Tech' category on the home page
-New getEventsForCategory() query to request all events for a given list of category ids from backend
NewsArticle relies on react hooks like useEffect(), so it needs to be rendered on the client side
-Basic page that displays all events in the backend database for a given Category ID
-Sidebar is now dynamically populated with categories fetched from the backend -Clicking a category on the sidebar brings up the Category-specific page powered by the new category-specific Page component
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Changes:
This closes #68