Skip to content

[IMP] Time Off: My time#18604

Closed
larm-odoo wants to merge 1 commit into
19.0from
19.0-time-off-update-my-time-larm
Closed

[IMP] Time Off: My time#18604
larm-odoo wants to merge 1 commit into
19.0from
19.0-time-off-update-my-time-larm

Conversation

@larm-odoo

Copy link
Copy Markdown
Contributor

Updating new pop-ups, including the warning, and new visual layout for My Time Off and My Allocations.

Original task card for this PR.

@robodoo

robodoo commented Jun 24, 2026

Copy link
Copy Markdown
Collaborator

Pull request status dashboard

@larm-odoo larm-odoo self-assigned this Jun 24, 2026
@larm-odoo larm-odoo added the 3 label Jun 24, 2026
@larm-odoo

Copy link
Copy Markdown
Contributor Author

Hi @theRealThagomizer - this is ready for a review. Thanks!

@C3POdoo C3POdoo requested a review from a team June 24, 2026 21:25

@theRealThagomizer theRealThagomizer left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hiya, @larm-odoo! Changes look good overall! I had a few suggestions and caught a few typos, but I think this is ready to move on otherwise.

Comment thread content/applications/hr/time_off/my_time.rst
Comment thread content/applications/hr/time_off/my_time.rst Outdated
Comment thread content/applications/hr/time_off/my_time.rst Outdated
Comment thread content/applications/hr/time_off/my_time.rst Outdated
Comment thread content/applications/hr/time_off/my_time.rst Outdated
@larm-odoo larm-odoo force-pushed the 19.0-time-off-update-my-time-larm branch from 4baf465 to cb7be1e Compare June 30, 2026 15:34
@larm-odoo larm-odoo requested a review from erjer-odoo June 30, 2026 15:34
@larm-odoo

Copy link
Copy Markdown
Contributor Author

Hi @erjer-odoo - this is ready for a final review. Thanks!

@erjer-odoo erjer-odoo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @larm-odoo -- just a few quick changes, thanks!

@robodoo delegate+

Comment thread content/applications/hr/time_off/my_time.rst Outdated
Comment thread content/applications/hr/time_off/my_time.rst Outdated
Comment thread content/applications/hr/time_off/my_time.rst Outdated
Comment thread content/applications/hr/time_off/my_time.rst Outdated
Comment thread content/applications/hr/time_off/my_time.rst Outdated
Comment thread content/applications/hr/time_off/my_time.rst Outdated
Comment thread content/applications/hr/time_off/my_time.rst Outdated
@larm-odoo larm-odoo force-pushed the 19.0-time-off-update-my-time-larm branch from cb7be1e to b74761d Compare June 30, 2026 17:50
@larm-odoo

Copy link
Copy Markdown
Contributor Author

@robodoo r+

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants