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

Registering users when admin approval is required #82

Open
claudiu-cristea opened this issue Jun 3, 2019 · 0 comments · May be fixed by #88
Open

Registering users when admin approval is required #82

claudiu-cristea opened this issue Jun 3, 2019 · 0 comments · May be fixed by #88
Labels
enhancement New feature or request

Comments

@claudiu-cristea
Copy link
Contributor

Currently, when a new user is register after a successful EU Login, if the site policy is to require admin approval, the account will be blocked. This should be configurable. For example some projects might consider that users registered via EU Login are safe and there's no need for approval.

Proposal

Add a new setting Block newly created users if the site requires admin approval on new account registering (find a better wording!). This should default to TRUE in order to ensure BC.
@brummbar brummbar added the enhancement New feature or request label Jun 3, 2019
claudiu-cristea added a commit that referenced this issue Jun 6, 2019
@claudiu-cristea
Issue #82: Add the new logical config.
70b428c
claudiu-cristea added a commit that referenced this issue Jun 6, 2019
@claudiu-cristea
Issue #82: Update existing site config.
aafa219
claudiu-cristea added a commit that referenced this issue Jun 6, 2019
@claudiu-cristea
Issue #82: Expose the new config in the settings form.
5cf4058
claudiu-cristea added a commit that referenced this issue Jun 6, 2019
@claudiu-cristea
Issue #82: Move the 'blocked user' test into its own feature and expa…
b801ee2 
…nd the test coverage.
claudiu-cristea added a commit that referenced this issue Jun 6, 2019
@claudiu-cristea
Issue #82: Make the subscriber aware of the new setting.
803ead4
@claudiu-cristea claudiu-cristea linked a pull request Jun 6, 2019 that will close this issue
Open
claudiu-cristea added a commit that referenced this issue Jun 6, 2019
@claudiu-cristea
Issue #82: Feature configure_authentication requires step definitions…
6d1dc81 
… from behat/mink-extension:^2.3.
claudiu-cristea added a commit that referenced this issue Jun 6, 2019
@claudiu-cristea
Issue #82: Feature configure_authentication requires step definitions…
51664f1 
… from behat/mink-extension:^2.3.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Optional user blocking when user register approval is required openeuropa/oe_authentication
2 participants
@claudiu-cristea @brummbar