-
Notifications
You must be signed in to change notification settings - Fork 1
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
[MINT-3201]/mto ready for review by #1520
base: feature/MINT-3175_mto
Are you sure you want to change the base?
[MINT-3201]/mto ready for review by #1520
Conversation
…ser account table
…m/CMSgov/mint-app into MINT-3201/mto_ready_for_review_by
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.
I chatted with @patrickseguraoddball about this today and we might want to consider some other approaches for this that might be a bit more friendly with change history
Functionally, I think this all works just fine and the resolvers/mutations seem to be in the right spots, but it just might be worth considering if we want to break out MTO Data like this into another table that's not on the model plan.
Let's sync up next week, but just leaving this comment so we can discuss further!
MINT-3201
Description
Also updated the linking table to be correctly reference the user account table
How to test this change
Verify that the unit tests work as expected.
PR Author Checklist
PR Reviewer Guidelines