Conversation
kaseyliu
left a comment
There was a problem hiding this comment.
Note (mostly for myself): you mentioned during standup but waiting on changing this to using Chakra UI components
|
I contacted Emi about it and she said not to worry about styling with Chakra UI components. Also decided it would be a hassle with installing the chakra ui components library, but if you want me to replace w chakra components, and youre fine w me editing package-lock, etc. I can do it. |
|
I'll make sure to get you in the loop next time or leave a note in the PR. |
There was a problem hiding this comment.
This looks great! Do you think you could change the Chakra stuff to just import it-- you shouldn't need to create components to put the code. I just added the Chakra provider. Sorry for not doing it sooner. I merged develop into your branch, and I think you should be able to just import the components from Chakra (I put an example button on the home page).
|
Chakra components in Navbar are already being used as described, to my knowledge. The components from Chakra setup, that appear when installing certain dependencies for the Chakra provider, have been removed. Please let me know if I'm misunderstanding something. |
kaseyliu
left a comment
There was a problem hiding this comment.
Looks great! I don't have any errors with the build on my end.


Developer: Nick Endresen
Closes #
Pull Request Summary
Create a Navbar header that has buttons that link to /home, /dashboard, and /profile, plus a button to sign out.
Modifications
Modified Navbar component and created page.tsx's for home and dashboard -- not profile.
Testing Considerations
Reviewer should verify the project runs cleanly in their development.
Pull Request Checklist
Screenshots/Screencast