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][FIX] web: show event date/duration on popover #166279

Conversation

fw-bot
Copy link
Contributor

@fw-bot fw-bot commented May 21, 2024

Issue:

  • when you open a calendar event popover, you can't see its time and duration

Steps To Reproduce:

  • In mobile view, go to calendar and create an event.
  • Click on the event and notice you can't see it's time and duration.

Solution:

  • We want to display the time in the popover for all modes. Therefore, I removed the scale === "month" condition from showTime. The showTime variable is used to set the isTimeHidden attribute, which is then checked in the computeDateTimeAndDuration function of the calendar popover.

opw-3829004


I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

Forward-Port-Of: #165793
Forward-Port-Of: #160907

@robodoo
Copy link
Contributor

robodoo commented May 21, 2024

Pull request status dashboard.

@fw-bot
Copy link
Contributor Author

fw-bot commented May 21, 2024

This PR targets saas-17.1 and is part of the forward-port chain. Further PRs will be created up to master.

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@robodoo robodoo added the forwardport This PR was created by @fw-bot label May 21, 2024
@fw-bot
Copy link
Contributor Author

fw-bot commented May 21, 2024

@kawkb @mcm-odoo ci/runbot failed on this forward-port PR

@C3POdoo C3POdoo added the OE the report is linked to a support ticket (opw-...) label May 21, 2024
Issue:
    - when you open a calendar event popover ,
    you can't see its time and duration

Steps To Reproduce:
    - Go to calendar and create an event.
    - Click on the event and notice you can't
    see it's time and duration in popover.

Solution:
    - We want to display the time in the popover for all
     modes. Therefore, I removed the scale === "month"
     condition from showTime. The showTime variable is used
     to set the isTimeHidden attribute, which is then checked
     in the computeDateTimeAndDuration function of the calendar
     popover.

opw-3829004

X-original-commit: c0017c2
@kawkb kawkb force-pushed the saas-17.1-saas-16.3-OPW-3829004-web_add_event_date_duration_on_mobile-kdeb-uSi9-fw branch from 200af82 to 0ab910d Compare May 22, 2024 10:04
@fw-bot
Copy link
Contributor Author

fw-bot commented May 22, 2024

@kawkb @mcm-odoo this PR was modified / updated and has become a normal PR. It should be merged the normal way (via @robodoo)

@kawkb
Copy link
Contributor

kawkb commented May 22, 2024

@robodoo r+

@C3POdoo C3POdoo requested review from a team, Iucapad and juliusc2066 and removed request for a team May 22, 2024 10:16
robodoo pushed a commit that referenced this pull request May 22, 2024
Issue:
    - when you open a calendar event popover ,
    you can't see its time and duration

Steps To Reproduce:
    - Go to calendar and create an event.
    - Click on the event and notice you can't
    see it's time and duration in popover.

Solution:
    - We want to display the time in the popover for all
     modes. Therefore, I removed the scale === "month"
     condition from showTime. The showTime variable is used
     to set the isTimeHidden attribute, which is then checked
     in the computeDateTimeAndDuration function of the calendar
     popover.

opw-3829004

closes #166279

X-original-commit: c0017c2
Signed-off-by: Michaël Mattiello (mcm) <mcm@odoo.com>
Signed-off-by: Kawtar Drissi El Bouzaidi (kdeb) <kdeb@odoo.com>
@robodoo robodoo closed this May 22, 2024
@fw-bot fw-bot deleted the saas-17.1-saas-16.3-OPW-3829004-web_add_event_date_duration_on_mobile-kdeb-uSi9-fw branch June 5, 2024 19:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
forwardport This PR was created by @fw-bot OE the report is linked to a support ticket (opw-...)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants