fix(client): merge and update the learn map (#36822)
This commit is contained in:
committed by
Ahmad Abdolsaheb
parent
a5b176be88
commit
e54a7fb350
@@ -30,8 +30,8 @@ describe('<NavLinks />', () => {
|
||||
it('renders to the DOM', () => {
|
||||
expect(root).toBeTruthy();
|
||||
});
|
||||
it('has 2 a tags', () => {
|
||||
expect(aTags.length === 2).toBeTruthy();
|
||||
it('has 3 a tags', () => {
|
||||
expect(aTags.length === 3).toBeTruthy();
|
||||
});
|
||||
|
||||
it('has link to portfolio', () => {
|
||||
|
@@ -26,7 +26,7 @@ const createOnClick = (navigate, isSignedIn) => e => {
|
||||
e.preventDefault();
|
||||
gtagReportConversion();
|
||||
if (isSignedIn) {
|
||||
return gatsbyNavigate('/');
|
||||
return gatsbyNavigate('/learn');
|
||||
}
|
||||
return navigate(`${apiLocation}/signin`);
|
||||
};
|
||||
|
@@ -5,6 +5,9 @@ export function NavLinks() {
|
||||
return (
|
||||
<div className='main-nav-group'>
|
||||
<ul className={'nav-list display-flex'} role='menu'>
|
||||
<li className='nav-theme' role='menuitem'>
|
||||
<Link to='/learn'>Projects</Link>
|
||||
</li>
|
||||
<li className='nav-theme' role='menuitem'>
|
||||
<Link to='/'>Light</Link>
|
||||
</li>
|
||||
|
Reference in New Issue
Block a user