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

Messy timezone mangling in website_event #27179

Closed
yajo opened this issue Sep 24, 2018 · 1 comment
Closed

Messy timezone mangling in website_event #27179

yajo opened this issue Sep 24, 2018 · 1 comment
Labels
9.0 10.0 11.0 Services Project, timesheet, helpdesk,...

Comments

@yajo
Copy link
Contributor

yajo commented Sep 24, 2018

Impacted versions: 9-12

Steps to reproduce:

  1. Today it's September 24th, 2018. This is the calendar for this week and the next one:
    captura de pantalla de 2018-09-24 09-30-19

  2. I'm Administrator, I live in Madrid:
    captura de pantalla de 2018-09-24 09-28-01

  3. I schedule an event in Sydney. It should be on October 1st, 2018, from 6 AM to 8 AM. I'm smart enough to compute mentally the tzdiff between Madrid and Sydney and I end up with this:
    captura de pantalla de 2018-09-24 09-33-31

  4. I publish the event in the website.

  5. I log out.

  6. I go to the website and check events for this week:
    captura de pantalla de 2018-09-24 09-35-42

Current behavior:

  1. Backend user needs to compute mentally the tzdiff between his current timezone and the event's.
  2. Frontend user sees Australian event displayed for this week, although it's scheduled for next week.

Expected behavior:

  1. Backend user should not need to be aware about his local timezone if the event is in a different one. It's important if he needs to schedule something in his calendar, but for managing the event (publishing it, managing tracks, publishing them, writing invitation emails...) he usually will only care about event's TZ.
  2. Frontend user should get events filtered only according to event's local timezone, as frontend users never care about a different timezone than the event's.

@Tecnativa

@pedrobaeza pedrobaeza added the Services Project, timesheet, helpdesk,... label Sep 24, 2018
yajo added a commit to Tecnativa/event that referenced this issue Sep 24, 2018
Before this patch, the user was getting the Public User's TZ on the
calendar view.

After merged, only the event's timzeone will be used to display dates.

It is, however, affected by odoo/odoo#27179,
which means that there is a possible mismatch between the date used
to search and the one used to display, most likely to be hit when
events have very different timzeones. Added to Known Issues.
Tardo pushed a commit to Tecnativa/event that referenced this issue May 13, 2019
Before this patch, the user was getting the Public User's TZ on the
calendar view.

After merged, only the event's timzeone will be used to display dates.

It is, however, affected by odoo/odoo#27179,
which means that there is a possible mismatch between the date used
to search and the one used to display, most likely to be hit when
events have very different timzeones. Added to Known Issues.
ajaniszewska-dev pushed a commit to ajaniszewska-dev/event that referenced this issue Sep 28, 2020
Before this patch, the user was getting the Public User's TZ on the
calendar view.

After merged, only the event's timzeone will be used to display dates.

It is, however, affected by odoo/odoo#27179,
which means that there is a possible mismatch between the date used
to search and the one used to display, most likely to be hit when
events have very different timzeones. Added to Known Issues.
ajaniszewska-dev pushed a commit to ajaniszewska-dev/event that referenced this issue Sep 29, 2020
Before this patch, the user was getting the Public User's TZ on the
calendar view.

After merged, only the event's timzeone will be used to display dates.

It is, however, affected by odoo/odoo#27179,
which means that there is a possible mismatch between the date used
to search and the one used to display, most likely to be hit when
events have very different timzeones. Added to Known Issues.
damdam-s pushed a commit to damdam-s/event that referenced this issue Jun 10, 2021
Before this patch, the user was getting the Public User's TZ on the
calendar view.

After merged, only the event's timzeone will be used to display dates.

It is, however, affected by odoo/odoo#27179,
which means that there is a possible mismatch between the date used
to search and the one used to display, most likely to be hit when
events have very different timzeones. Added to Known Issues.
damdam-s pushed a commit to damdam-s/event that referenced this issue Jun 10, 2021
Before this patch, the user was getting the Public User's TZ on the
calendar view.

After merged, only the event's timzeone will be used to display dates.

It is, however, affected by odoo/odoo#27179,
which means that there is a possible mismatch between the date used
to search and the one used to display, most likely to be hit when
events have very different timzeones. Added to Known Issues.
chandni299 pushed a commit to elego/event that referenced this issue Nov 15, 2021
Before this patch, the user was getting the Public User's TZ on the
calendar view.

After merged, only the event's timzeone will be used to display dates.

It is, however, affected by odoo/odoo#27179,
which means that there is a possible mismatch between the date used
to search and the one used to display, most likely to be hit when
events have very different timzeones. Added to Known Issues.
chandni299 pushed a commit to elego/event that referenced this issue Dec 14, 2021
Before this patch, the user was getting the Public User's TZ on the
calendar view.

After merged, only the event's timzeone will be used to display dates.

It is, however, affected by odoo/odoo#27179,
which means that there is a possible mismatch between the date used
to search and the one used to display, most likely to be hit when
events have very different timzeones. Added to Known Issues.
Camille0907 pushed a commit to Camille0907/event that referenced this issue Dec 16, 2022
Before this patch, the user was getting the Public User's TZ on the
calendar view.

After merged, only the event's timzeone will be used to display dates.

It is, however, affected by odoo/odoo#27179,
which means that there is a possible mismatch between the date used
to search and the one used to display, most likely to be hit when
events have very different timzeones. Added to Known Issues.
@C3POdoo
Copy link
Contributor

C3POdoo commented Mar 30, 2023

Dear @yajo,

Thank you for your report but we are closing it due to inactivity.
We apology if we could not look at your request in time.
If your report still makes sense, don't hesitate to reopen a new one. We will try to check it as soon as possible.

This is an automated message.

@C3POdoo C3POdoo closed this as completed Mar 30, 2023
Camille0907 pushed a commit to Camille0907/event that referenced this issue Jun 7, 2023
Before this patch, the user was getting the Public User's TZ on the
calendar view.

After merged, only the event's timzeone will be used to display dates.

It is, however, affected by odoo/odoo#27179,
which means that there is a possible mismatch between the date used
to search and the one used to display, most likely to be hit when
events have very different timzeones. Added to Known Issues.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
9.0 10.0 11.0 Services Project, timesheet, helpdesk,...
Projects
None yet
Development

No branches or pull requests

4 participants