-
Notifications
You must be signed in to change notification settings - Fork 16
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
Create user tables for basic login.gov auth #2716
Comments
@chouinar Do I have this right >> The UUID for table I guess is the user table only going to reference login.gov users, not test users for example. |
The user table has no knowledge of anything to do with login.gov - it's a very generic table we'll add to later. The link_external_user table is where we put info that will connect a user to an oauth service, in this case, just login.gov The lk_external_user_type is just to store enums for different types of external users (in this case, just login.gov) in our DB |
## Summary Fixes #{[2716](#2716)} ### Time to review: __10 mins__ ## Changes proposed 3 user tables migration script updated factories to create new users ## Context for reviewers Users tables will be used for Oauth2 ## Additional information > Screenshots, GIF demos, code examples or output to help show the changes working as expected. --------- Co-authored-by: nava-platform-bot <[email protected]> Co-authored-by: Michael Chouinard <[email protected]> Co-authored-by: kai [they] <[email protected]> Co-authored-by: David Dudas <[email protected]>
Summary
We want to create three tables in our database:
user
lk_external_user_type
This follows our usual pattern for lookup tables
The enum we create for this will only have a single value of
login_gov
at this time.link_external_user
Note that we have been using integers for a lot of our tables primary keys up to this point. However we want these to be UUIDs so they aren't just incrementally counting upward. If you make the python type
Mapped[uuid.UUID]
it should make the DB type correctly, but we'll need to validate that (it should also automatically populate the ID if none provided, we'll likely need to configure that).Acceptance criteria
The text was updated successfully, but these errors were encountered: