Skip to content
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

Navbar username/sign out display on login, code cleanup #8

Merged
merged 1 commit into from
Apr 8, 2022

Conversation

Luberski
Copy link
Contributor

@Luberski Luberski commented Apr 7, 2022

  • Adjust fonts to figma design
  • [Login page] Redirect user to Home page on succesful log in
  • Add logout button to navbar
  • Display username in navbar when user is logged in

@Luberski Luberski requested review from xdk78, Ryczko and pmatyjasik April 7, 2022 18:12
<Link to={'/'} className="flex items-center text-zinc-600">
<div className="heart mr-4"></div>
<Link to={'/'} className="flex items-center text-zinc-600 font-serif text-2xl">
{/* <div className="heart mr-4"></div> */}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:trollface:


useEffect(() => {
if (user) {
console.log('user', user);
navigate('/');
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@@ -1,4 +1,4 @@
@import url('https://fonts.googleapis.com/css2?family=Inter:wght@400;500;700;800;900&display=fallback');
@import url('https://fonts.googleapis.com/css2?family=Montserrat:wght@400;500;700&family=Pacifico&display=swap');
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💯

}
body {
@apply text-zinc-900;
@apply text-zinc-900 bg-zinc-100;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💪

@xdk78 xdk78 merged commit 963b282 into dev Apr 8, 2022
@xdk78 xdk78 deleted the welcome_page branch April 8, 2022 10:41
Ryczko pushed a commit that referenced this pull request Feb 8, 2023
Navbar username/sign out display on login, code cleanup
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants