-
Notifications
You must be signed in to change notification settings - Fork 24.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] Calendar: Improved recurrent event when dst happens #15202
Conversation
cd3eaef
to
145d4de
Compare
145d4de
to
3b8acc5
Compare
3b8acc5
to
a221d90
Compare
a221d90
to
3bae0bf
Compare
8921f5d
to
aca4d95
Compare
ce2c34a
to
62a8170
Compare
62a8170
to
2d74cbc
Compare
2d74cbc
to
ec9181a
Compare
1d50f56
to
24b3b23
Compare
24b3b23
to
7021449
Compare
7021449
to
77f2863
Compare
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.
Not working (anymore?)
@@ -139,6 +139,7 @@ | |||
{'no_create_edit': True}"/> | |||
<field name="alarm_ids" widget="many2many_tags" /> | |||
<field name="location" /> | |||
<field name="event_tz" /> |
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.
Hide if not recurring event, else it will be confusing
@@ -528,7 +528,8 @@ def _prepare_holidays_meeting_values(self): | |||
'stop': self.date_to, | |||
'allday': False, | |||
'state': 'open', # to block that meeting date in the calendar | |||
'privacy': 'confidential' | |||
'privacy': 'confidential', | |||
'event_tz': self.user_id.tz, |
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.
Is it required since a holiday is not a recurring event?
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.
yes, we should add the employees timezone when creating the event. It's done upon validation, which is not done by the employee but by a manager. Unless we make the assumption the manager shares the same timezone it could be wrong.
2973f73
to
533bd3e
Compare
533bd3e
to
9292227
Compare
The daylight savings time (summer/winter time) is taken into account when creating a recurrent event via calendar which causes problem in time.
e.g:
Whenever you create an event in Brussels timezone for ex. it goes either 1 hour prior or 1 hour after scheduled event time in case of daylight saving time(in March, October).