-
-
Notifications
You must be signed in to change notification settings - Fork 829
Fix: Unable to verify email address error #3528
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! I am a bit surprised we didn't seem to hit this in testing...
const ErrorDialog = sdk.getComponent("dialogs.ErrorDialog"); | ||
const ErrorDialog = sdk.getComponent("dialogs.ErrorDialog"); | ||
if (err.errcode === 'M_THREEPID_AUTH_FAILED') { | ||
Modal.createTrackedDialog("E-mail hasn't been verified yet", "", ErrorDialog, { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In our strings, we consistently use "email" instead of "e-mail", so please change all these to "email".
as it's confusing to have nothing happen at all when clicking continue straight away.
unsure why this code seems duplicated?
9716855
to
1ab19be
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for working on this! Feel free to merge after fixing strings and re-running i18n.
if (err.errcode === 'M_THREEPID_AUTH_FAILED') { | ||
Modal.createTrackedDialog("Email hasn't been verified yet", "", ErrorDialog, { | ||
title: _t("Your email address hasn't been verified yet"), | ||
description: _t("Click the link in the email you received to verify, " + |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since there's no subject after the "and", it's better to drop the "," according to the strange rules of English.
description: _t("Click the link in the email you received to verify, " + | |
description: _t("Click the link in the email you received to verify " + |
if (err.errcode === 'M_THREEPID_AUTH_FAILED') { | ||
Modal.createTrackedDialog("E-mail hasn't been verified yet", "", ErrorDialog, { | ||
title: _t("Your email address hasn't been verified yet"), | ||
description: _t("Click the link in the email you received to verify, " + |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
description: _t("Click the link in the email you received to verify, " + | |
description: _t("Click the link in the email you received to verify " + |
Fixes element-hq/element-web#11040
Also added a dialog when server says that e-mail hasn't been verified yet.