Feature/cgd 47

[JaneMoroz]

Description and impacts

• Add the dark & light mode functionality
• Add the dark & light mode toggle button to navbar

Jira related story

CGD-47

Additional info

I used next-themes.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
chingu-dashboard	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Sep 19, 2023 11:59am

[Dan-Y-Ko]

1. Can you check your colors in the theme again. It looks like the secondary-content, accent, and accent-content colors aren't correct but also do a check again in case I missed a mistake somewhere else as well.
2. I think this would be a good time to add some tests. I think for now just a simple unit test is fine. Write a test for the functionality of the theme toggle. Create a test folder in the components folder and put your test file in there with .test.tsx extension. Don't need barrel file for this since we're not going to need to access the test file anywhere else in the project.

[Dan-Y-Ko]

There were some problems with the theming identified, so it needs to be changed. So you can hold off on updating the colors for now. I'll let you know when it's finalized.

[JaneMoroz]

2. unit test

I think cypress runs tests that are only in cypress/e2e folder 🤔

[Dan-Y-Ko]

2. unit test

I think cypress runs tests that are only in cypress/e2e folder 🤔

you can use react testing library for the unit tests and cypress for the e2e tests.

[Dan-Y-Ko]

As previously discussed, the changes to the color of the theming can be handled in a different pr. This one looks good to me.

[JaneMoroz]

I have noticed that the profile image path is wrong, I'll fix it when I get home.

[marktlinn]

LGTM, just need to fix the avatar image import which @JaneMoroz has already referenced. After that it's good to merge.

[JaneMoroz]

LGTM, just need to fix the avatar image import which @JaneMoroz has already referenced. After that it's good to merge.

Done!