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

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

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

Conversation

lase-odoo
Copy link

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

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
@robodoo
Copy link
Collaborator

robodoo commented Dec 17, 2024

Pull request status dashboard

@lase-odoo
Copy link
Author

community PR: odoo/odoo#190867

@lase-odoo
Copy link
Author

Followup of the discussion of the first proposition: odoo/odoo#189363

@C3POdoo C3POdoo requested a review from a team December 18, 2024 12:00
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

Successfully merging this pull request may close these issues.

2 participants