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.

Sign up for GitHub

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

Jump to bottom

Userdetails returns userId in userName field for some users #898

Open
chrisala opened this issue Dec 7, 2023 · 0 comments
Open

Userdetails returns userId in userName field for some users #898

chrisala opened this issue Dec 7, 2023 · 0 comments
Assignees
@chrisala
Labels
technical-issue Flag for an issue that doesn't have direct UAT/functional testing requirements

Comments

@chrisala
Copy link
Collaborator

chrisala commented Dec 7, 2023 •
edited
Loading

This is likely related to the cognito switch but the effect is that it causes emails to be unable to be sent for users which get their userId in the userName field.
MERIT is relying on the userName field for the email for legacy reasons.
@chrisala chrisala self-assigned this Dec 7, 2023
chrisala added a commit that referenced this issue Dec 7, 2023
@chrisala
Explicitly map userName to email field from userdetails #898
8b20dc1
salomon-j added a commit that referenced this issue Dec 7, 2023
@salomon-j
Merge pull request #899 from AtlasOfLivingAustralia/feature/issue898
19236d5 
Explicitly map userName to email field from userdetails #898
@chrisala chrisala added the technical-issue Flag for an issue that doesn't have direct UAT/functional testing requirements label Dec 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@chrisala chrisala

Labels
technical-issue Flag for an issue that doesn't have direct UAT/functional testing requirements
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@chrisala