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

Admin's password recoverable behavior and recovery workflow #48

Merged
merged 10 commits into from
Nov 15, 2016

Conversation

danguita
Copy link
Contributor

@danguita danguita commented Nov 15, 2016

This PR implements #16.

Before reviewing

This PR depends on #46, so let's try to get it fully merged first. The last commit is covering the whole feature actually.

What does this PR do?

It implements the conventional Admin password recovery workflow, which behaves this way:

  1. The new password form asks for an email address to match an active Admin from http:https://gobierto.dev/admin/admin/passwords/new.
  2. After that, an email with a new generated token is sent to the matching Admin user.
  3. The system is exposing a URL to catch the token param and ask the Admin for new user credentials (password and password confirmation in this case).
  4. The system logs in the user automatically and revokes the token that has just been used.

How should this be manually tested?

Let's check out http:https://gobierto.dev/admin/admin/passwords/new and follow the whole process.

@danguita danguita changed the title Admin's recoverable behavior and recovery workflow Admin's password recoverable behavior and recovery workflow Nov 15, 2016
@ferblape ferblape merged commit ac2c26b into master Nov 15, 2016
@danguita danguita deleted the 16-admin-password-recovery branch November 15, 2016 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants