Skip to content

[19.0][ADD] project_hr_expense_analytic - #1785

Merged
OCA-git-bot merged 1 commit into
OCA:19.0from
camptocamp:19-project_hr_expense_analytic
Aug 24, 2026
Merged

OCA-git-bot merged 1 commit into
OCA:19.0from
camptocamp:19-project_hr_expense_analytic

Conversation

@simahawk

@simahawk simahawk commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

This module links expense records to a project and, optionally, a task,
and derives the expense's analytic distribution from that project.

@OCA-git-bot OCA-git-bot added series:19.0 mod:project_hr_expense_analytic Module project_hr_expense_analytic labels Aug 5, 2026

@imlopes imlopes 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.

LGTM

@gurneyalex gurneyalex left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

preapproving, the i18n needs fixing IMO

Comment thread project_hr_expense_analytic/i18n/fr.po
class ProjectProject(models.Model):
_inherit = "project.project"

hr_expense_ids = fields.One2many("hr.expense", "project_id", string="HR Expenses")

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

is this used anywhere?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I don't think so. We could get rid of it.

@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

Co-authored-by: Denis Leemann <denis.leemann@camptocamp.com>
Co-authored-by: Italo Lopes <dev@nomadiplus.com>
@simahawk
simahawk force-pushed the 19-project_hr_expense_analytic branch from e87e8d4 to 22a4f2b Compare August 18, 2026 07:09
@simahawk

Copy link
Copy Markdown
Contributor Author

/ocabot merge nobump

@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 19.0-ocabot-merge-pr-1785-by-simahawk-bump-nobump, awaiting test results.

@OCA-git-bot
OCA-git-bot merged commit ae5e23e into OCA:19.0 Aug 24, 2026
7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at fc80a79. Thanks a lot for contributing to OCA. ❤️

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants