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

[IMP] purchase reports extensibility #25903

Closed

Conversation

MiquelRForgeFlow
Copy link
Contributor

Description of the issue/feature this PR addresses:

In a custom module, suppose you extend the purchase report template using inherit. Then if you want to add <tr> tags it's difficult if there is a <tr t-foreach= and you are forced to do a xpath replace.

To avoid that, this PR adds the same trick as sale report.

Current behavior before PR:
Purchase reports lack some extensibility.

Desired behavior after PR is merged:
Extensibility of purchase reports are improved.

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

@MiquelRForgeFlow
Copy link
Contributor Author

@mart-e @Yenthe666 I don't know who to ping here, can you help?

@mart-e mart-e added the Logistics about stock, mrp, delivery, purchase label May 29, 2019
@MiquelRForgeFlow
Copy link
Contributor Author

Handled with 5618298.

@MiquelRForgeFlow MiquelRForgeFlow deleted the master-fix-purchase-reports branch July 26, 2019 11:10
@robodoo robodoo added seen 🙂 CI 🤖 Robodoo has seen passing statuses labels Jul 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI 🤖 Robodoo has seen passing statuses Logistics about stock, mrp, delivery, purchase
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants