search

argenos / nldates-obsidian

Work with dates in natural language in Obsidian
MIT License
490 stars 44 forks source link

Not working reliably in some timezones: @thursday - @sunday gives last week's date #90

Open mitchmeister opened 2 years ago

mitchmeister commented 2 years ago

We are 1300+ UTC here and I'm finding that some phrases like "@next monday" (which should be "2022-03-07") are failing ("2022-02-28). Both "@next monday" and "@monday" are "2022-02-28". Even "@sunday" is showing "2022-02-27" (& "Start of week" setting = Monday)

mitchmeister commented 2 years ago

trying just now (on Mon 2022-03-07): @wednesday gave me 2022-03-09 (correct) but @thursday gave me 2022-03-03

mitchmeister commented 2 years ago

still buggy. Any plan to fix this?

mitchmeister commented 2 years ago

Just clarifying (& reminding you of this bug). @mon - @wed provide dates from this week.
@thu - @sun provide dates from previous week. (@next thu - @next sun provide dates from this week)

Am I missing something or is this a bug?

(My timezone is +1200 GMT/UTC. "Week starts on" set to Monday)

argenos commented 2 years ago

Thanks for all the reminders. Does this comment explain the behaviour you're seeing? https://github.com/argenos/nldates-obsidian/issues/96#issuecomment-1099126026

still buggy. Any plan to fix this?

Currently, I don't have time to dive into this, but if you want to tackle it, feel free to discuss your proposal here and then open a PR for it.

dredhorse commented 2 years ago

I added some ideas to the mentioned #96. The point is though: THIS atm isn't working correctly. Today is Tuesday and my THIS Thursday is last week 😞