-
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
Fix update user resolver to exclude branches as it is only changed by super admin via other resolvers #665
Conversation
… super admin via other resolvers
Visit the preview URL for this PR (updated for commit 533821a): https://sistering-dev--pr665-rickson-edit-profile-1vpf3ga1.web.app (expires Tue, 06 Sep 2022 15:59:19 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 |
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.
Getting an error when I click the "Save Changes" button in the edit account page for volunteer. @MatoPlus Can you see if this is related to your changes?
Tested locally with Prisma Studio, everything works great! One question not of concern though- when creating accounts (in |
This is correct. It doesn't hurt to leave it in for a more complete "create" resolver. |
Sounds good, LGTM! |
Ticket link
Closes #664
Implementation description
Steps to test
What should reviewers focus on?
Checklist