-
Notifications
You must be signed in to change notification settings - Fork 111
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
[C-4455] Update otp email #8727
Conversation
|
e134815
to
1da19d3
Compare
<!-- | ||
<![endif]--> | ||
</div> | ||
const getOtpEmail = ({ otp, expire, copyrightYear }) => { |
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.
How plug-n-play was this from Figma? Did you just have to interpolate the values, and the rest was copy pasta?
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.
mostly copy and paste, some small things we need to look out for in general like the copyright year, some links should open in a new tab, some inline links are missing in the template
[fa19828] [PAY-3091] Move social handles out of identity into dn via entity manager (#8728) Saliou Diallo [fe60b30] Fix dupe swagger id (#8745) JD Francis [ed8febd] [C-4455] Update otp email (#8727) Isaac Solo [8496557] [C-4263] Add OptionsFilterButton and implement more search filters (#8737) Sebastian Klingler [49b5a56] [C-4660] POC: Pre-fetch CN stream urls (no /stream redirects) (#8741) JD Francis [dfb04fd] on call: fix search on stage (#8743) alecsavvy
Description
Update otp email to match new design system.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide repro instructions & any configuration.
Ran on stage