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

[17.0][MIG] hr_expense_cancel: Migration to version 17.0 #238

Merged
merged 17 commits into from
Jul 30, 2024

Conversation

vvrossem
Copy link

@vvrossem vvrossem commented May 29, 2024

Supersedes #233

@vvrossem vvrossem force-pushed the 17.0-mig-hr_expense_cancel branch from 536d3da to aa53fc6 Compare May 29, 2024 13:13
stefan-tecnativa and others added 6 commits June 13, 2024 09:34
The multi own account test has been removed as it is no longer possible
in odoo core to register the payment of several expense sheet reports at
the same time.

Co-Authored By:
Stefan Ungureanu
aiendry-aktivsoftware
In this version, if a product doesn't have cost, you are only able
to enter total amount, so the test should be adjusted to this
condition.
@vvrossem vvrossem force-pushed the 17.0-mig-hr_expense_cancel branch from aa53fc6 to db17c35 Compare June 13, 2024 07:36
@OCA-git-bot
Copy link
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). 🤖

@Saran440
Copy link
Member

/ocabot migration hr_expense_cancel

@OCA-git-bot OCA-git-bot added this to the 17.0 milestone Jul 18, 2024
@OCA-git-bot OCA-git-bot mentioned this pull request Jul 18, 2024
9 tasks
@gurneyalex
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 17.0-ocabot-merge-pr-238-by-gurneyalex-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit cddf9b4 into OCA:17.0 Jul 30, 2024
6 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 39abb02. 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.