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

Make 'create user' a bit more robust #2144

Merged
merged 2 commits into from
Nov 18, 2020
Merged

Make 'create user' a bit more robust #2144

merged 2 commits into from
Nov 18, 2020

Conversation

bobdenotter
Copy link
Member

No description provided.

@bobdenotter
Copy link
Member Author

Pinging @I-Valchev.. I'm merging this in, because i want to prep the release, but tomorrow i'd like to hear your opinion on the weirdness that's going on here.. I don't see how this could ever have worked correctly, and why it chose yesterday to start breaking on this.

@bobdenotter bobdenotter merged commit 3411384 into 4.1 Nov 18, 2020
@bobdenotter bobdenotter deleted the fix/create-user branch November 18, 2020 13:42
@@ -151,6 +152,7 @@ Feature: Users & Permissions
And I should see "Suggested secure password"

@javascript
@foo
Copy link
Member

Choose a reason for hiding this comment

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

@bobdenotter Oops... think we have to remove that! 🤪

Copy link
Member Author

Choose a reason for hiding this comment

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

Removed it! 🙈

@I-Valchev
Copy link
Member

I-Valchev commented Nov 19, 2020

@bobdenotter I don't know why it was breaking? Or what was breaking?

It's strange that we need the default locale there, because the form is submitted and there the select already has a prepopulated option to begin with. Same for status.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants