Skip to content

[18.0][MIG] sale_order_tag: Migration to 18.0#5

Open
natuan9 wants to merge 13 commits into18.0from
18.0-mig-sale_order_tag
Open

[18.0][MIG] sale_order_tag: Migration to 18.0#5
natuan9 wants to merge 13 commits into18.0from
18.0-mig-sale_order_tag

Conversation

@natuan9
Copy link
Owner

@natuan9 natuan9 commented Mar 25, 2025

Change:

  • field_parent is removed, it was unused by the web client starting from this commit

patrickrwilson and others added 12 commits March 25, 2025 17:21
This module adds tags to sales order. The tags are for independant to sales orders and not related to the opportunities like the other tag field that exists.

[IMP] domain fix and better readme description

Update __manifest__.py

Co-authored-by: Bhavesh Odedra <bodedra@opensourceintegrators.com>

Update tag string

Co-authored-by: Bhavesh Odedra <bodedra@opensourceintegrators.com>

[FIX] Readme LINT

[FIX] Reviewer suggestions

- Removed plural on  saleordertag class
- Moved menu item to proper xml
- Added company_id field to tags with rule
- Moved sequence to data
- Additional code cleanup

[FIX] Typo
Currently translated at 100.0% (20 of 20 strings)

Translation: sale-workflow-14.0/sale-workflow-14.0-sale_order_tag
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-14-0/sale-workflow-14-0-sale_order_tag/it/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: sale-workflow-16.0/sale-workflow-16.0-sale_order_tag
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-16-0/sale-workflow-16-0-sale_order_tag/
Currently translated at 100.0% (19 of 19 strings)

Translation: sale-workflow-16.0/sale-workflow-16.0-sale_order_tag
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-16-0/sale-workflow-16-0-sale_order_tag/it/
License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).-->
<odoo>
<record id="sale_order_tag_view_list" model="ir.ui.view">
<field name="name">sale.order.tag.tree</field>

Choose a reason for hiding this comment

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

I think we should revert it

@natuan9 natuan9 force-pushed the 18.0-mig-sale_order_tag branch from 0e40f83 to cabcc56 Compare March 28, 2025 03:17
@natuan9 natuan9 requested a review from chaule97 March 28, 2025 03:18
License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).-->
<odoo>
<record id="sale_order_tag_view_list" model="ir.ui.view">
<field name="name">sale.order.tag.tree</field>

Choose a reason for hiding this comment

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

Suggested change
<field name="name">sale.order.tag.tree</field>
<field name="name">sale.order.tag.list</field>

@natuan9 natuan9 force-pushed the 18.0-mig-sale_order_tag branch from cabcc56 to 103ff66 Compare March 28, 2025 04:01
@natuan9 natuan9 requested a review from chaule97 March 28, 2025 04:01
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.

10 participants