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

External Login registration without password #3488

Open
wants to merge 5 commits into
base: dev
from

Conversation

Projects
None yet
2 participants
@MichaelPetrinolis
Copy link
Contributor

commented Apr 19, 2019

Add a setting to register external users without requesting a password
image
Generate a username instead of parsing claims from external login info
Integrate Recaptcha in External Login Registration
image
Integrate Email Confirmation for users created from External Logins

@MichaelPetrinolis

This comment has been minimized.

Copy link
Contributor Author

commented May 2, 2019

I must check all the scenarios after merging with dev Seems ok

@sebastienros

This comment has been minimized.

Copy link
Member

commented May 3, 2019

The PR shows all the changes since you created the PR, can you rebase it on top of dev?

@MichaelPetrinolis MichaelPetrinolis force-pushed the MichaelPetrinolis:extLogin_registration branch from a6feeb1 to 99b5119 May 3, 2019

MichaelPetrinolis added some commits Apr 18, 2019

Add a setting to register external users without requiring a password
Use common logic to verify a login
Use the unverified email address rule for the external users
WIP
send confirmation email on external registration
integrate recaptcha on external registration
move BaseEmailController's code to an extension class to use the regi…
…stration from external login

fix RegistrationControllerTests
remove RoleNames from call to CreateUserAsync
when registering a new user

@MichaelPetrinolis MichaelPetrinolis force-pushed the MichaelPetrinolis:extLogin_registration branch from 99b5119 to 5f536f4 May 3, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.