Skip to content

Conversation

@fw-bot
Copy link
Collaborator

@fw-bot fw-bot commented Dec 16, 2021

The suggested folder structure does not comply with guidelines:
https://www.odoo.com/documentation/15.0/developer/misc/other/guidelines.html#file-naming

Concerning printable reports which contain mainly data preparation and Qweb templates naming is
the following :

addons/plant_nursery/
|-- report/
|   |-- plant_order_reports.xml (report actions, paperformat, ...)
|   |-- plant_order_templates.xml (xml report templates)

Forward-Port-Of: #1350

The suggested folder structure does not comply with guidelines:
https://www.odoo.com/documentation/15.0/developer/misc/other/guidelines.html#file-naming

X-original-commit: 34bc61a
@robodoo
Copy link
Collaborator

robodoo commented Dec 16, 2021

@fw-bot
Copy link
Collaborator Author

fw-bot commented Dec 16, 2021

Ping @poma-odoo, @AntoineVDV
This PR targets master and is the last of the forward-port chain containing:

To merge the full chain, say

@fw-bot r+

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@AntoineVDV
Copy link
Collaborator

@fw-bot r+

robodoo pushed a commit that referenced this pull request Dec 16, 2021
… naming

The suggested folder structure does not comply with guidelines:
https://www.odoo.com/documentation/15.0/developer/misc/other/guidelines.html#file-naming

> Concerning printable reports which contain mainly data preparation and Qweb templates naming is
the following :

    addons/plant_nursery/
    |-- report/
    |   |-- plant_order_reports.xml (report actions, paperformat, ...)
    |   |-- plant_order_templates.xml (xml report templates)

closes #1426

Forward-port-of: #1350
Signed-off-by: Antoine Vandevenne (anv) <anv@odoo.com>
@robodoo robodoo closed this Dec 16, 2021
@robodoo robodoo temporarily deployed to merge December 16, 2021 10:29 Inactive
@robodoo robodoo added the 15.2 label Dec 16, 2021
@fw-bot fw-bot deleted the master-poma-odoo-rd-J_reports-fcLu-fw branch December 30, 2021 10:46
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