-
Notifications
You must be signed in to change notification settings - Fork 0
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.
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
feat: create endpoint to get users #81
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
god job
FYI, I think @Luana-ns have some changes to be pushed here to make this work mode "native" to nextjs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to be careful with the things we're changing, as they are also used on the App UI, not only on the admin.
Also, let's study a little more about:
- next.js
- zod
- how
raise
works
I left a few links for reference
Hey @marco-souza we solved most issues here, just need to enhance some final points, let you know when this is ready |
Description
This PR fully implements the
GET Users
route, it includes:GET /users
fetchUsers
DAO methodTODO
I plan to finish this on my weekend
Demo
@ogatalars @Luana-ns please also review this PR, comment doubts and suggestions.