-
-
Notifications
You must be signed in to change notification settings - Fork 305
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
[IMP]base_ubl_payment: Add payment means to SEPA direct debit/credit … #861
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks so good to me 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code review, Makes sense 😆
@oca/maintainers, can we get a merge? 🙏 |
@alexis-via Is this one ok to you? |
@etobella Can you please merge 🙏 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/ocabot merge nobump
@etobella Is the oca bot sleeping?? 🤖 ? |
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at 023bed2. Thanks a lot for contributing to OCA. ❤️ |
…transfer
Fixes:
When creating a peppol invoice.
Screenshot from:
https://test.peppolautoriteit.nl/validate
By adding: