Skip to content

Comments

[IMP] Employees - Departments#16175

Closed
larm-odoo wants to merge 1 commit into19.0from
19.0-employees-departments-larm
Closed

[IMP] Employees - Departments#16175
larm-odoo wants to merge 1 commit into19.0from
19.0-employees-departments-larm

Conversation

@larm-odoo
Copy link
Contributor

No appraisal templates on form, expenses changed to reports so the text changed. Colors also changed form squares to circles.

Original task card for this PR.

@robodoo
Copy link
Collaborator

robodoo commented Jan 28, 2026

Pull request status dashboard

@larm-odoo larm-odoo added the 2 label Jan 28, 2026
@larm-odoo larm-odoo self-assigned this Jan 28, 2026
@larm-odoo larm-odoo requested a review from erjer-odoo January 28, 2026 21:59
@larm-odoo
Copy link
Contributor Author

Hi @erjer-odoo - this is ready for a peer review, thanks!

@C3POdoo C3POdoo requested a review from a team January 28, 2026 22:01
Copy link
Contributor

@erjer-odoo erjer-odoo left a comment

Choose a reason for hiding this comment

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

Hi @larm-odoo ! This LGTM -- I only noticed one missing period in the output.

@larm-odoo larm-odoo force-pushed the 19.0-employees-departments-larm branch from 6b3caa0 to a2cb1d6 Compare January 29, 2026 13:59
@larm-odoo larm-odoo requested a review from Felicious January 29, 2026 14:02
@larm-odoo
Copy link
Contributor Author

Hi @Felicious - this is ready for a final review!

@larm-odoo larm-odoo requested a review from nikibrown January 30, 2026 15:01
@larm-odoo
Copy link
Contributor Author

Hi @nikibrown - per Felicia, this is ready for a tech review - thanks!

Copy link
Contributor

@nikibrown nikibrown left a comment

Choose a reason for hiding this comment

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

@larm-odoo LGTM 🚀

@nikibrown
Copy link
Contributor

@robodoo delegate+

@larm-odoo
Copy link
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