Add expiry message for email change confirm #439
Conversation
|
This pull request is being automatically deployed with Vercel (learn more). egghead-io-nextjs – ./🔍 Inspect: https://vercel.com/eggheadio/egghead-io-nextjs/1sdall3t6 egghead-next-storybook – ./🔍 Inspect: https://vercel.com/eggheadio/egghead-next-storybook/o02t9qvea |
| @@ -87,7 +87,6 @@ const CopyToClipboard: FunctionComponent<{ | |||
| <IconLink className="w-5" /> | |||
| <span>Copy link</span> | |||
| </button> | |||
| <Toaster position="bottom-center" /> | |||
There was a problem hiding this comment.
Moving this to a shared location in _app.tsx.
| toast.success( | ||
| "You've successfully updated your email address", | ||
| ) | ||
| router.replace('/user') | ||
| } | ||
| } catch (e) { | ||
| toast.error( | ||
| 'This link for changing your email has been used or has expired. Feel free to request a new link.', | ||
| {duration: 6000, icon: '❌'}, | ||
| ) |
There was a problem hiding this comment.
Introduce success and error toasts for Email Change Confirmation to make it clearer that it succeeded or failed.
jbranchaud
force-pushed
the
jb/add-expiry-message-for-email-change-confirm
branch
from
2f2fd10
to
4faa23c
Compare
|
suggestion(follow-up): I think this message should contain more details about what is happening including the email addresses that are being changed.
|
kodiakhq
bot
deleted the
jb/add-expiry-message-for-email-change-confirm
branch
We had someone revisiting a used link and trying to re-confirm several times. This message should help them know they've already used it (or that it has expired). It uses the react-hot-toast package.
CleanShot.2021-02-15.at.11.31.09.mp4