Skip to content

Conversation

@larm-odoo
Copy link
Contributor

Updating to new standards.

Original task card for this PR.

@robodoo
Copy link
Collaborator

robodoo commented Nov 3, 2025

Pull request status dashboard

@larm-odoo larm-odoo self-assigned this Nov 3, 2025
@larm-odoo larm-odoo added the 3 label Nov 3, 2025
@larm-odoo larm-odoo requested a review from jero-odoo November 3, 2025 21:25
@C3POdoo C3POdoo requested a review from a team November 3, 2025 21:27
@larm-odoo
Copy link
Contributor Author

Hi @jero-odoo - this is ready for a peer review. Thanks!

Copy link
Contributor

@jero-odoo jero-odoo left a comment

Choose a reason for hiding this comment

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

Approved with a few comments. Also, it looks like there is an additional image in the folder (expense-status.png) that you removed from the file but did not delete the image. :)

@larm-odoo larm-odoo force-pushed the 18.0-expenses-update-reports-larm branch from 6903d9b to b807e9f Compare November 5, 2025 19:47
@larm-odoo larm-odoo requested a review from Felicious November 5, 2025 19:47
@larm-odoo
Copy link
Contributor Author

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

Copy link
Contributor

@Felicious Felicious left a comment

Choose a reason for hiding this comment

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

@Felicious Felicious changed the title [IMP] Expenes: Updating expense reports [IMP] Expenses: Updating expense reports Nov 6, 2025
@Felicious
Copy link
Contributor

@robodoo r+

@robodoo
Copy link
Collaborator

robodoo commented Nov 6, 2025

This PR is already reviewed, reviewing it again is useless.

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.

5 participants