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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix creating automatic nicknames when taken by user_groups #9578

Merged
merged 1 commit into from
Jul 18, 2022

Conversation

microstudi
Copy link
Contributor

@microstudi microstudi commented Jul 14, 2022

馃帺 What? Why?

Similar as #9452 this prevents generating nicknames already taken by user_groups. This feature is currently used when the user is created by an OAuth login but i might become more relevant in the future if the nickname is removed in the new design.

馃搶 Related Issues

Link your PR to an issue

Testing

Describe the best way to test or validate your PR.

馃摲 Screenshots

Please add screenshots of the changes you're proposing
Description

鈾ワ笍 Thank you!

@request-info
Copy link

request-info bot commented Jul 14, 2022

It seems like you didn't give us much information about what you're trying to do here. We would appreciate it if you could provide us with more info about this issue/PR!

Copy link
Contributor

@ahukkanen ahukkanen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

馃憤

I think this is a no-brainer, so I'll just merge it even though you also asked review from Andr茅s.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
module: core type: fix PRs that implement a fix for a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants