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 confirmation instructions recovery behavior and workflow #49

Merged
merged 2 commits into from
Nov 15, 2016

Conversation

danguita
Copy link
Contributor

This PR implements #16.

Before reviewing

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

What does this PR do?

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

  1. The confirmation instructions recovery form asks for an email address to match an active Admin from https://gobierto.dev/admin/admin/confirmations/new.
  2. After that, the confirmation token is regenerated and the confirmation email is immediately sent to the Admin so a new email confirmation process has just started.

How should this be manually tested?

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

@ferblape ferblape merged commit faffc3b into master Nov 15, 2016
@danguita danguita deleted the 16-admin-resend-confirmation 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.

2 participants