zooniverse / Panoptes-Front-End

Front end for zooniverse/Panoptes
https://www.zooniverse.org
Apache License 2.0
64 stars 75 forks source link

Email Settings: add "valid email" and "verified email" status #7083

Closed shaunanoordin closed 1 month ago

shaunanoordin commented 2 months ago

PR Overview

Staging branch URL: https://pr-7083.pfe-preview.zooniverse.org Towards #7032

This PR adds "🟢 valid / 🔴 invalid email" and "🟢 verified / 🔴 unverified email" indicators to the Email Settings page (under User -> Settings -> Email)

Screenshot 2024-04-24 at 02 55 55

Status

Ready for review.

Next steps: the "request confirmation email" functionality requires either

  1. a fairly custom (and complex) fetch() function inside email.jsx to POST to panoptes.zooniverse.org/api/users/confirmation with the auth token inside the code, or
  2. an update to PJC to do the same, or
  3. as a placeholder, a simple link to https://panoptes.zooniverse.org/users/confirmation/new

For this PR, the "request confirmation email" has been disabled. (I got halfway through option 1 but ran into some bugs due to a malformed request, and option 3 hit a small snag when I realised I need to check the env to get the correct URL.)

coveralls commented 2 months ago

Coverage Status

coverage: 56.994% (+0.01%) from 56.98% when pulling 6125950ee6aebbed35bf0c4975e74ca5c22d338b on email-verification-status into b8cf174c780f7b5a242b91529115cfa7235b3742 on master.

shaunanoordin commented 1 month ago

Thank you! 👍 I'll address the following feedback before I merge:

The "request confirmation email" functionality will appear in a follow up PR.