Skip to content
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

PB-1106: Use cognito's preferred username #43

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

msom
Copy link
Contributor

@msom msom commented Nov 26, 2024

Introduces a new user_id field in access.User, which is an autogenerated 12-digit nanoid. This user_id serves as the Cognito username, while the existing access.User.username is used as the preferred_username in Cognito.

The database migration will assign a user_id to all existing users.

To update existing Cognito users, you can use ./manage.py cognito_sync. This process will delete the old users in Cognito and re-create them using the new user_id.

Checklist

@msom msom requested a review from benschs November 26, 2024 07:57
@msom msom force-pushed the feat-PB-1106-preferred-username branch from 3e6a543 to 2491dc1 Compare November 26, 2024 08:28
Pipfile Outdated Show resolved Hide resolved
app/tests/utils/test_short_id.py Outdated Show resolved Hide resolved
@msom msom force-pushed the feat-PB-1106-preferred-username branch 2 times, most recently from 270bcfb to f4ff0b7 Compare November 26, 2024 14:55
@msom msom force-pushed the feat-PB-1106-preferred-username branch from f4ff0b7 to 271bf0f Compare November 27, 2024 08:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants