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

[MIG][17.0] ebill_postfinance #736

Merged
merged 19 commits into from
Jan 28, 2025

Conversation

TDu
Copy link
Member

@TDu TDu commented Sep 17, 2024

No description provided.

@TDu TDu force-pushed the 17.0-mig-ebill_postfinance branch 2 times, most recently from 8d178b2 to 6668cfb Compare September 17, 2024 14:45
@yvaucher
Copy link
Member

/ocabot migration ebill_postfinance

@OCA-git-bot OCA-git-bot added this to the 17.0 milestone Sep 30, 2024
@OCA-git-bot OCA-git-bot mentioned this pull request Sep 30, 2024
3 tasks
Copy link
Member

@yvaucher yvaucher left a comment

Choose a reason for hiding this comment

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

LGTM waiting on dependencies (CI error)

TDu and others added 10 commits January 16, 2025 12:23
Remove the dependency on stock in the `ebill_postfinance` module.
And add a new module `ebill_postfinance_stock` that will integrate
in the xml invoice informations related to the deliveries.

This is for Odoo implementation that sell only services and have no
stock to manage. And do not use the `stock` module.
Now that `account_invoice_export` does not requires it anymore.
@vvrossem vvrossem force-pushed the 17.0-mig-ebill_postfinance branch from c12743e to 75b3857 Compare January 16, 2025 11:24
@ecino
Copy link

ecino commented Jan 28, 2025

Dependencies are merged. Can you rebase your branch to retrigger the build and see if it passes?

@TDu TDu force-pushed the 17.0-mig-ebill_postfinance branch from 75b3857 to a615241 Compare January 28, 2025 12:06
@TDu TDu force-pushed the 17.0-mig-ebill_postfinance branch from a615241 to e166d77 Compare January 28, 2025 12:10
@simahawk
Copy link

/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-736-by-simahawk-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 96d2735 into OCA:17.0 Jan 28, 2025
7 checks passed
@OCA-git-bot
Copy link
Contributor

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