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][FW] [14.0][FIX] account_payment_partner: Error when salesperson tries to print invoices #1370

Conversation

sebalix
Copy link
Contributor

@sebalix sebalix commented Oct 23, 2024

Port of the following PR from 14.0 to 17.0:

The following PRs have been blacklisted:

CarlosRoca13 and others added 2 commits October 23, 2024 10:59
…invoices

This error is happening because the salesperson has not access to payment_method_id.

With this changes we avoid the error and the invoices are printed like normally.

TT31296
@sebalix sebalix marked this pull request as ready for review October 23, 2024 09:23
@sebalix sebalix changed the title [17.0][FW] [14.0][FW] account_payment_partner: multiple ports from 13.0 [17.0][FW] [14.0][FW] [FIX] account_payment_partner: Error when salesperson tries to print invoices Oct 23, 2024
@sebalix sebalix changed the title [17.0][FW] [14.0][FW] [FIX] account_payment_partner: Error when salesperson tries to print invoices [17.0][FW] [14.0][FIX] account_payment_partner: Error when salesperson tries to print invoices Oct 23, 2024
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.

Fast-tracking this easy one:

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 17.0-ocabot-merge-pr-1370-by-pedrobaeza-bump-patch, awaiting test results.

@pedrobaeza pedrobaeza added this to the 17.0 milestone Oct 23, 2024
@OCA-git-bot OCA-git-bot merged commit e3b86af into OCA:17.0 Oct 23, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

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

4 participants