-
-
Notifications
You must be signed in to change notification settings - Fork 436
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
Fix wrong payment info template paths #1201
Fix wrong payment info template paths #1201
Conversation
We could discuss if this was intended to have the possibility to use different layout styles. But defining a default existing template is better than having no template for pdf |
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.
LGTM
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.
Tested.
Unit Test Results1 files 1 suites 0s ⏱️ Results for commit c7d316a. |
I think this change caused a regression in rendering payment information as PDF. This change causes the toPdf methods to use the HTML templates instead of the PDF templates. This causes invalid PDF rendering for the payment methods. The PDF template files do exist: As far as I know, there is no option for including a PDF attachment on invoice emails. Was the submitter using an add-on module for email attachments? Perhaps there is a bug in the add-on? |
Description (*)
In some Payment block classes the 'toPdf' function sets a wrong template path. These templates are present in another folder.
Manual testing scenarios (*)
Contribution checklist (*)