feat: Feature Request - Color theme chooser

[RezaZR]

@mazipan I added color theme chooser but not heavily focus on color, just a simple theme chooser that you can change between light and dark.

[vercel]

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/mazipan/buku-saku-pramuka/35uxdo1m9
✅ Preview: https://buku-saku-pramuka-git-master.mazipan.vercel.app

[mazipan]

Ini preview nya belum jalan nih mas @RezaZR.
Saya gak bisa lihat hasilnya.
Minta tolong coba trigger commit biar force trigger vercel preview.

[mazipan]

Sama ada pengetahuan yang pengen saya share, ini sepele tapi membantu kalau kontribusi ke Open Source di Github.

1. Jangan pakai branch utama master atau main untuk PR, PR ini bisa dilihat hasilnya. Changes dari PR sebelumnya kebawa ke PR ini kan ya? Ini bikin PR nya gak bersih mas. Menimbulkan kesan kalau kita asal-asalan.

2. Buat branch sendiri, dan kasih naming sesuai fitur/bug. Misal: rezazr/add-color-theme, tiap satu PR ya satu branch, jangan di reuse.

3. Kalau terbiasa dengan git rebase, bisa coba pakai interactive rebase untuk squash commit nya, jadi rapi. Kelihatan cuma ada satu commit, meskipun sebelumnya kita kerjanya bikin banyak commit.

4. Pasang auto sync untuk forked repo, saya biasa pakai https://github.com/apps/pull. Jadi kalau ada update di master saya, fork repo punya mas nya bisa dapat update nya, meskipun delay.

🙏👍

[arispati]

saya juga mengerjakan ini, dan sudah selesai, mau PR ternyata sudah ada yg PR 😢
saya sudah gak bisa PR tuk yang ini ya mas @mazipan atau masih bisa?

[mazipan]

Itulah kenapa, sebaiknya sebelum kerja di reserve dulu issue nya 😂
Jadi bisa satu siapa yang lagi ngerjain 👍

Kalau begini sih gak enak juga, sudah ada yang bikin PR duluan soalnya.
Saya akan tetep coba PR yang pertama ini sampai bisa di merge ya mas 🙏

No offense loh ya 🙏

Sama-sama belajar aja.
Atau nanti bisa bantu-bantu aja cek hasilnya.
Kalau ada yang bisa di-improve lagi ya boleh banget lah

[arispati]

ok mas, buat pengalaman juga sih 👍

baru handle juga project NextJS, jadi agak lama bikinnya, aslinya developer laravel & react native 😸

[mazipan]

Asik lah mas, sesekali enak kan lari dari kebiasaan sehari-hari?

Tapi boleh juga mas, open PR aja branch nya.
Sayang udah di coding.
Biar sama-sama kita review aja buat belajar

[arispati]

boleh mas @mazipan kalau untuk preview, tp tetap prioritaskan PR yg ini saja, soalnya belum tahu best pratice buat theme switcher di NextJS jadi coding ala kadarnya

ijin PR ya mas @RezaZR for preview only 👍

[RezaZR]

@mazipan halo mas, maaf baru bisa nyentuh komputer lagi, untuk link preview memang tidak ada perubahan tapi button view deployment di pojok kanan bawahnya jika diklik terdapat perubahan. Untuk branch lain kali saya akan pakai branch mas.

@arispati silahkan mas 😃