Skip to content

Conversation

@luc-demeyer
Copy link
Contributor

This commits adds the possibility to update the payment reference on a payment transaction before generating XML file.

@luc-demeyer
Copy link
Contributor Author

forward port of #1170

@pedrobaeza pedrobaeza changed the title [IMP] update payment reference on open payment order [17.0][IMP] account_payment_order: update payment reference on open payment order Apr 30, 2024
@pedrobaeza pedrobaeza added this to the 17.0 milestone Apr 30, 2024
@pedrobaeza
Copy link
Member

Please put the commit message properly according guidelines:

https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#commit-message

@luc-demeyer luc-demeyer force-pushed the 17-update-payment-reference-on-open-payment-order branch from ab437fa to 3d0fef8 Compare April 30, 2024 16:48
@pedrobaeza
Copy link
Member

@luc-demeyer can you rebase and forced push for having CI?

… order

This commits adds the possibility to update the payment reference on a
payment transaction before generating XML file.
@luc-demeyer luc-demeyer force-pushed the 17-update-payment-reference-on-open-payment-order branch from 3d0fef8 to 8faf8ed Compare June 9, 2024 20:10
@luc-demeyer
Copy link
Contributor Author

@pedrobaeza : rebased

Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

The same as v16, so merging it:

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 17.0-ocabot-merge-pr-1257-by-pedrobaeza-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit dbaf831 into OCA:17.0 Jun 10, 2024
@OCA-git-bot
Copy link
Contributor

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

3 participants