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

[12.0][MIG] hr_timesheet_overtime_rate_begin_end #38

Open
wants to merge 7 commits into
base: 12.0
Choose a base branch
from

Conversation

carmenbianca
Copy link
Member

Description

Backport of #33

Depends on OCA/timesheet#693

Odoo task (if applicable)

T12629

Checklist before approval

  • Tests are present (or not needed).
  • Credits/copyright have been changed correctly.
  • Change log snippet is present.
  • (If a new module) Moving this to OCA has been considered.

@carmenbianca carmenbianca force-pushed the 12.0-mig-hr_timesheet_overtime_begin_end branch from f493fe5 to d861ab0 Compare July 18, 2024 09:47
@carmenbianca carmenbianca changed the title [12.0][MIG] hr_timesheet_overtime_begin_end [12.0][MIG] hr_timesheet_overtime_rate_begin_end Jul 29, 2024
@carmenbianca carmenbianca changed the title [12.0][MIG] hr_timesheet_overtime_rate_begin_end [12.0][MIG] hr_timesheet_rate_overtime_begin_end Sep 3, 2024
@carmenbianca carmenbianca force-pushed the 12.0-mig-hr_timesheet_overtime_begin_end branch from cbdf715 to ed28ba9 Compare September 3, 2024 13:45
@carmenbianca carmenbianca changed the title [12.0][MIG] hr_timesheet_rate_overtime_begin_end [12.0][MIG] hr_timesheet_overtime_rate_begin_end Sep 3, 2024
@carmenbianca carmenbianca force-pushed the 12.0-mig-hr_timesheet_overtime_begin_end branch from ed28ba9 to 3e7a0e4 Compare September 27, 2024 14:54
This is the name of the module in v16. Because this module is new, this
will save us some small migratory work later.

Signed-off-by: Carmen Bianca BAKKER <[email protected]>
@carmenbianca carmenbianca force-pushed the 12.0-mig-hr_timesheet_overtime_begin_end branch from 3e7a0e4 to 5dd155a Compare September 27, 2024 15:03
Signed-off-by: Carmen Bianca BAKKER <[email protected]>
@huguesdk
Copy link
Member

@carmenbianca would you please take a look at the failing test?

@carmenbianca
Copy link
Member Author

@huguesdk depends on OCA/timesheet#693

@huguesdk
Copy link
Member

/ocabot merge nobump

@github-grap-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 12.0-ocabot-merge-pr-38-by-huguesdk-bump-nobump, awaiting test results.

github-grap-bot added a commit that referenced this pull request Oct 11, 2024
Signed-off-by huguesdk
@github-grap-bot
Copy link
Contributor

Congratulations, your PR was merged at 26f78c0. Thanks a lot for contributing to coopiteasy. ❤️

@carmenbianca
Copy link
Member Author

@huguesdk you re-opened this PR. is that correct?

@huguesdk
Copy link
Member

huguesdk commented Oct 11, 2024

yes, it is presently unmerged (i force-pushed 12.0 to unmerge it), since it depends on the unmerged change in the oca’s timesheet repo.

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.

3 participants