When the user has already set an email address, we shouldn't display an input to let them change it. We already do the right thing when their email address is set and confirmed, but not when it's set and unconfirmed.
Checked in betalabs - all works as exepcted.
@MMiller_WMF - the email field of unconfirmed email becomes non-modifiable from the Help panel (incl the Ask question on Mentorship). The text under the text field is the same as before - "You'll receive an email notificaiton...". I wonder if it's understandable for a user that they still need to confirm their email address.
|present production - unconfirmed email||betalabs - unconfirmed email|
Actually, @Etonkovidova, I just realized that we should double-check the EventLogging. This seems like the sort of change that has the potential to break EventLogging for this part of the workflow. Would you be willing to try that out? To make sure that the correct events continue to come through for these steps?
I checked betalabs eventlogging (the most recent timestamp - 20190613141120) for three schemas - HelpPanel, HomepageModule, and HomepageVisit. The change of emails never was actually tracked; the state of the email address is being tracked. I re-checked recorded actions - and there were no discrepancies.