fix: weekday check function for other locales

onesine / react-tailwindcss-datepicker

Modern date range picker component for React using Tailwind 3 and dayjs. Alternative to Litepie Datepicker.

MIT License

552 stars 170 forks source link

Closed cheeselemon closed 3 months ago

cheeselemon commented 11 months ago

Fixing issue #150 , #221

Screenshot 2023-12-07 at 6 57 08 PM

Problem:

Reason:

This is due to getNumberOfDay being stuck at number = 0, when delcaring dayjs.locale() other than "en" globally outside library.

The Fix:

Instead of using hard-coded weekday string array ["Sunday", ... "Saturday"], a function is created to specifically use previously set locales.

Maycon-Santos commented 8 months ago

Merge please! 🥲

onesine commented 3 months ago

Hello @cheeselemon 👋. Sorry for the late reply. Thanks for the contribution.