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

[FW][IMP] marketing: match the Display Timezone description with tooltips #11820

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fw-bot
Copy link
Collaborator

@fw-bot fw-bot commented Jan 22, 2025

Issue:

The current descriptions and naming of the Date fields (date_begin, date_end) and the Timezone field (date_tz) can be confusing in the form view of the event model. Specifically:

The timezone used for converting the form dates to the database is determined by the context and not by the Timezone field present on the form. However, this Timezone field determines the timezone used for displaying the event's date/time on the website. This change clarifies the situation according to the update done in the community PR

opw-4323142

Forward-Port-Of: #11749
Forward-Port-Of: #11596

Issue:

The current descriptions and naming of the `Date` fields (`date_begin`,
`date_end`) and the `Timezone` field (`date_tz`) can be confusing in the
form view of the event model. Specifically:

The timezone used for converting the form dates to the database is
determined by the context and not by the `Timezone` field present on the
form. However, this `Timezone` field determines the timezone used
for displaying the event's date/time on the website. This change
clarifies the situation according to the update done in the community PR

opw-4323142

X-original-commit: fe9d186
@robodoo
Copy link
Collaborator

robodoo commented Jan 22, 2025

Pull request status dashboard

@jeh-odoo
Copy link

@robodoo r+

@robodoo
Copy link
Collaborator

robodoo commented Jan 22, 2025

@jeh-odoo you can't review+.

@lase-odoo
Copy link

robodoo r+

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

Successfully merging this pull request may close these issues.

4 participants