Skip to content

Conversation

@poma-odoo
Copy link
Contributor

@poma-odoo poma-odoo commented Dec 3, 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)

@robodoo
Copy link
Collaborator

robodoo commented Dec 3, 2021

@AntoineVDV AntoineVDV changed the base branch from 15.0 to master December 9, 2021 13:48
@AntoineVDV AntoineVDV changed the base branch from master to 15.0 December 9, 2021 13:49
@C3POdoo C3POdoo requested a review from a team December 9, 2021 13:49
@AntoineVDV AntoineVDV changed the title [fix] Reports tutorial differ [FIX] developer/rdtraining: comply with the guidelines for report naming Dec 16, 2021
Copy link
Collaborator

@AntoineVDV AntoineVDV left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@robodoo r+ squash

@robodoo
Copy link
Collaborator

robodoo commented Dec 16, 2021

Merge method set to squash

@robodoo robodoo closed this in 34bc61a Dec 16, 2021
@robodoo robodoo temporarily deployed to merge December 16, 2021 09:39 Inactive
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 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 #1425

Forward-port-of: #1350
Signed-off-by: Antoine Vandevenne (anv) <anv@odoo.com>
@fw-bot fw-bot deleted the poma-odoo-rd-J_reports branch December 30, 2021 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants