Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove Demo page and API router #406

Closed
taesungh opened this issue Dec 2, 2024 · 0 comments · Fixed by #506
Closed

Remove Demo page and API router #406

taesungh opened this issue Dec 2, 2024 · 0 comments · Fixed by #506
Assignees
@cyriljoby

Comments

@taesungh
Copy link
Member

taesungh commented Dec 2, 2024

Similar to #56/#57 and following from #41 and #72, we've verified the API works properly even with data fetching in server components, so we should remove the demo page in Demo.tsx and the corresponding API router in demo.py. As written, the add-user endpoint will break the expected database schema for the Users collection.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cyriljoby cyriljoby

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

deleted Demo.tsx and its router demo.py HackAtUCI/irvinehacks-site
2 participants
@taesungh @cyriljoby