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
Security section routing + set first password #65
Conversation
This pull request is being automatically deployed with Vercel (learn more). connect-account – ./🔍 Inspect: https://vercel.com/fewlines/connect-account/qemhun2aj connect-account-storybook – ./🔍 Inspect: https://vercel.com/fewlines/connect-account-storybook/ijv07h3g5 |
6cff47b
to
735ad8f
Compare
735ad8f
to
0f9ec69
Compare
0f9ec69
to
4db9c54
Compare
ebc481e
to
51e1687
Compare
src/components/display/fewlines/DesktopNavigationBar/DesktopNavigationBar.tsx
Show resolved
Hide resolved
src/components/display/fewlines/MobileNavigationBar/MobileNavigationBar.tsx
Show resolved
Hide resolved
…fewlinesco/connect-account into CU-64kz08-security_routing_set_password
export const Security: React.FC<SecurityProps> = ({ isPasswordSet }) => { | ||
return ( | ||
<ShadowBox> | ||
<Link href={`/account/security/${isPasswordSet ? "update" : "set"}`}> |
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.
Why do we need two URLs?
Is the answer, we did but we don't anymore? 😇
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.
Daaaamn nice catch. I chose to default it as update
.
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
This PR aims at adding the second section to Account: Security.
Context
The security seconds will have three functions:
What was done in this PR:
AccountPage
(i.e. sections page) integration tests.Type of change
Issues
No issues so far.
How Has This Been Tested?
With integration tests.
Screenshots
Checklist: