-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
fix(dav): Prioritize timezone from core/login #41866
Conversation
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
/backport to stable28 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice
'core', | ||
'timezone', | ||
); | ||
if ($fromConfig !== '') { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wanted to say that the value might not be a valid timezone ID, but someone fixed that a couple of months ago. #36001 🙈
Older values might still be bad though, as I didn't add a repair step.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good enough IMO. The value is set at every login.
Summary
The DAV app tries to guess a user's timezone from the calendar timezones and the availability settings. It turns out Nextcloud already tracks a timezone, set when a user logs in. Let's use that as primary source, then fall back to calendar timezones and availability timezone.
How to test
Master: calendar timezone is used
Here: login timezone is used
In the long run we should probably only set, not update the timezone at login, and make the timezone configurable via the personal settings.
Checklist