ArslanYM / StarterHive

πŸš€βœ¨ Help beginners to contribute to open source projects of remotely hiring organizations.
https://starter-hive.vercel.app/
MIT License
206 stars 200 forks source link

Issue #321 light mode removed #328

Closed Henrique-mazzoleni closed 9 months ago

Henrique-mazzoleni commented 9 months ago

This is a PR for the issue #321

I have removed every light mode option throughout the project.

However, I have not removed the Theme Context. I Haven't deleted the Switch Component although it will no longer be rendered on the screen.

Should I remove these two files from the code as well?

welcome[bot] commented 9 months ago

πŸ’–Thanks for opening this pull request! Please check out our contributing guidelines.πŸ’–

vercel[bot] commented 9 months ago

The latest updates on your projects. Learn more about Vercel for Git β†—οΈŽ

Name Status Preview Comments Updated (UTC)
starter-hive βœ… Ready (Inspect) Visit Preview πŸ’¬ Add feedback Sep 12, 2023 1:52pm
Henrique-mazzoleni commented 9 months ago

oh, just remembered I ran the format script on the main folder. Let me revert those changes.

ArslanYM commented 9 months ago

oh, just remembered I ran the format script on the main folder. Let me revert those changes.

Also you need to resolve some conflicts with the main branch. You can also consider removing the toggle and theme Context as they are no longer required.

Henrique-mazzoleni commented 9 months ago

@ArslanYM Resolved the conflicts and removed the theme context. I think it's in order now.

welcome[bot] commented 9 months ago

Congrats on merging your first pull request! We here at StarterHive are proud of you!πŸŽ‰πŸŽ‰πŸŽ‰ alt text