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

Email Verification: 2 issues; refresh un-verifies, verify email button should always be available. #3438

Closed
kcondon opened this issue Oct 27, 2016 · 15 comments · Fixed by #7516

Comments

@kcondon
Copy link
Contributor

kcondon commented Oct 27, 2016

Users are starting to use this feature and one reported an issue where they had confirmed their email but the account page said not verified and they did not know what to do, see RT 242896

I found a couple things that may impact this:

  1. If I'm on the account info page where there is a verify email button, I press it and then after clicking the confirmation button I refresh that page, it asks to resubmit the form and then it "unverifies" me and sends another email.
  2. It seems the verify email button does not appear until after the initial confirmation email timeout, default 24hrs. In cases where there is an issue, like this, it may be useful to always display the button.
@pdurbin
Copy link
Member

pdurbin commented Oct 28, 2016

I just noticed this bug at #3405 (comment)

@bsilverstein95 bsilverstein95 self-assigned this Oct 28, 2016
@bsilverstein95
Copy link
Contributor

Assigning this issue to myself, will work to recreate both of these and go from there. May need to consult the team regarding best practice with changing the Verify Email button's behavior (keeping it visible after confirmation vs having it disappear).

@mheppler
Copy link
Contributor

Proposed solution:

  • Always show "Verify Email" button on Account pg
  • User clicks button, backend logic...
    • checks for and clears expired tokens
    • checks for active tokens
    • if none, send email with new token
    • if active token found, triggers confirmation popup telling them to check their email

@chewsw
Copy link

chewsw commented Oct 9, 2018

May I check if there are any updates on the missing email verification button? I've encountered cases where the email verification button do not appear even after the 24 hours deadline has passed.

@pdurbin
Copy link
Member

pdurbin commented Oct 9, 2018

@chewsw hi, no one is working on this bug. If someone out there in the community would like to work on it, I'm happy to point out where in the code to start looking.

@djbrooke djbrooke added Status: Ready and removed Feature: Account & User Info Help Wanted: Code Mentor: pdurbin UX & UI: Design This issue needs input on the design of the UI and from the product owner Type: Bug a defect User Role: Sysadmin Installs, upgrades, and configures the system, connects via ssh Status: Ready labels Oct 24, 2018
@djbrooke djbrooke removed their assignment Nov 9, 2020
@djbrooke
Copy link
Contributor

I'd like to take another pass at this as part of a sprint. I was just getting back to things on Sept 8th when we discussed above (thanks @TaniaSchlatter and @pdurbin) and I asked for it to be moved back to Needs Discussion, but reviewing this again I think we can talk through the remaining issues after picking this up. My goal with this issue is to fix the verification system so that users can reliably put their accounts into a "verified" status, not make any changes to the privileges of verified/non-verified users. That can come next.

@sekmiller sekmiller self-assigned this Dec 10, 2020
sekmiller added a commit that referenced this issue Jan 12, 2021
sekmiller added a commit that referenced this issue Jan 12, 2021
sekmiller added a commit that referenced this issue Jan 12, 2021
sekmiller added a commit that referenced this issue Jan 14, 2021
@sekmiller
Copy link
Contributor

UI change for review. Part of the solution to this bug was to always display the "Verify Email" button while the user's email is unverified. If the user clicks the button while there is an active token this popup is rendered:

Screen Shot 2021-01-14 at 11 11 03 AM

@sekmiller
Copy link
Contributor

For reference. this is the existing success message when the user clicks "Verify Email" when there is no active token:

Screen Shot 2021-01-14 at 10 26 04 AM

@mheppler mheppler self-assigned this Jan 15, 2021
mheppler added a commit that referenced this issue Jan 19, 2021
@mheppler
Copy link
Contributor

Added responsive layout clean up to the fix the overlapping of the email address and verified status label, as reported in #4484. If the containing width is too small, the button and/or status label will wrap to another line. (See screenshot.)

Screen Shot 2021-01-19 at 3 24 35 PM

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 a pull request may close this issue.

9 participants