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
#486 users backend refactoring #511
Conversation
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.
nice work!
…ortheastern-Electric-Racing/FinishLine into #486-users-backend-refactoring
Let me know if I need to change any of the stub data, wasn't sure what to change there tbh. Also could you explain what the jest.Spyons are doing in the tests, I know they're mocking returns from prisma, but I'm not specifically sure how/why |
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.
awesome! pretty small stuff left
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.
yahoo
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.
yahoo2
Notes
This is me taking a first pass at this based on what I saw in the risks PR. There are definitely some places where I need some help(what to return for updateUserSettings and verification I'm doing the userLogin ones correctly). I'm not really sure what you meant by #7 in the ticket, I wasn't sure where the userId was a param I shouldn't change/ where it's a userId I should replace with the auth userId. Also I still need to do the test stuff, but wanted to get what I have done up first.
To Do
Update user tests and user test data
Checklist
It can be helpful to check the
Checks
andFiles changed
tabs.Please review the contributor guide and reach out to your Tech Lead if anything is unclear.
Please request reviewers and ping on slack only after you've gone through this whole checklist.
yarn.lock
changes (unless dependencies have changed)Closes #486