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
Adds login button to verify account page #255
Conversation
This pull request has been linked to Shortcut Story #14239: Add Login Button to Confirmation Page. |
|
||
import Logo from '@/components/common/Logo'; | ||
|
||
function LandingHeader() { | ||
const location = useLocation(); |
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.
const location = useLocation(); | |
const location = useLocation(); | |
const isConfirmationPage = location.pathname === APP_PATH.VERIFY_PAGE |
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.
APP_PATH
isn't listed in the paths file, so I added /verify-account
to ROUTES
.
return ( | ||
<nav className="border-b border-primary-800 py-8"> | ||
<div className="container mx-auto flex flex-wrap items-center justify-between"> | ||
<Logo /> | ||
<div className="space-x-8"> | ||
{login && ( |
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.
{login && ( | |
{isConfirmationPage && ( |
Scope of changes
Adds a log in button to the header that should only appear on the verify-account page, after a user has completed the registration process.
Fixes SC-14239
Type of change
Acceptance criteria
https://www.awesomescreenshot.com/video/15164591?key=b5ddc0cc7cfd878515285309d786fecc
Author checklist
Reviewer(s) checklist