-
Notifications
You must be signed in to change notification settings - Fork 66
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
Mobile: Update demographics policy and update all associated controllers and … #12813
Conversation
I changed a couple of things outside of our module on behalf of @tpharrison for the mobile team members wondering why there are changes outside of mobile |
I had to change one swagger test @tpharrison let me know if I should have fixed it another way |
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.
Looks good. Thanks for making these updates!
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.
Love to see this cross team collaboration. Using the demographics policy as the source of truth is a nice improvement. I'm not exactly sure why this allowed us to simplify the spec setup, but I'm happy to see that too. Nice improvements.
🚀
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.
Appreciate all the reviews!
…service to use policy
Summary
We have confirmation on who should be able to update gender identity and preferred name. The policy is now updated to reflect that
Related issue(s)
Testing done
Updated all tests to ensure they still pass
What areas of the site does it impact?
Preferred name and gender updating
Acceptance criteria
Requested Feedback