Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Intervals.icu syncing workouts / timezone issue #251

Open
quietvoid opened this issue Feb 25, 2025 · 6 comments
Open

Intervals.icu syncing workouts / timezone issue #251

quietvoid opened this issue Feb 25, 2025 · 6 comments

Comments

@quietvoid
Copy link

quietvoid commented Feb 25, 2025

Hello.

I noticed there's an issue with planned workouts when the day is different on the server side.
It's currently 20H EST and a workout made for today (2025-02-24) does not sync.

However if I copy it into tomorrow, then it does show up.
I'm thinking there's a difference between the timezones and the code decided the workout isn't for the current day.

Maybe the local timezone should be detected?

I'm seeing a request for /api/intervals/events?oldest=2025-02-25&newest=2025-02-25 so the date is not right client side.

@quietvoid
Copy link
Author

It seems isoDate is kinda broken. https://github.com/dvmarinoff/Auuki/blob/master/src/utils.js#L59
The offseted date isn't used at all, so it just always returns local date to UTC.

@dvmarinoff
Copy link
Owner

Hey thanks for taking time to report this issue!

quietvoid added a commit to quietvoid/Auuki that referenced this issue Feb 26, 2025
@dvmarinoff
Copy link
Owner

dvmarinoff commented Feb 26, 2025

Hey I added a potential fix e55ab3e on the develop branch which is deployed on dev.auuki.com, you can login there with the same creds. Hope the fix is as easy as that, but timezones usually have many surprises.

@quietvoid
Copy link
Author

The commit doesn't fix it, the offset is still ignored since the return never uses d.
See #253.

@dvmarinoff
Copy link
Owner

Another attempt with 03849e5

@quietvoid
Copy link
Author

That looks like it should work, I'll try it out tonight.
Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants