Skip to content

[19.0][MIG] sale_order_tag#4258

Open
eugenios73 wants to merge 18 commits intoOCA:19.0from
Studio73:19.0-mig-sale_order_tag
Open

[19.0][MIG] sale_order_tag#4258
eugenios73 wants to merge 18 commits intoOCA:19.0from
Studio73:19.0-mig-sale_order_tag

Conversation

@eugenios73
Copy link
Copy Markdown
Contributor

[19.0][MIG] sale_order_tag: Migration from 18.0 to 19.0

patrickrwilson and others added 17 commits April 7, 2026 17:42
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/
Currently translated at 100.0% (19 of 19 strings)

Translation: sale-workflow-18.0/sale-workflow-18.0-sale_order_tag
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-18-0/sale-workflow-18-0-sale_order_tag/it/
@rousseldenis
Copy link
Copy Markdown
Contributor

/ocabot migration sale_order_tag

@rousseldenis
Copy link
Copy Markdown
Contributor

rousseldenis commented Apr 9, 2026

@eugenios73 Thanks for this. Cannot pre-commit changes be applied in a separate commit ?

Copy link
Copy Markdown
Contributor

@Reyes4711-S73 Reyes4711-S73 left a comment

Choose a reason for hiding this comment

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

LGTM
Functional review OK

@eugenios73 eugenios73 force-pushed the 19.0-mig-sale_order_tag branch from c1c387d to 17a7be5 Compare April 9, 2026 08:48
@eugenios73
Copy link
Copy Markdown
Contributor Author

@eugenios73 Thanks for this. Cannot pre-commit changes be applied in a separate commit ?

Hi @rousseldenis. Thanks for you're review. No, the reason is for not any change has applied in pre-commit command execution.

@OCA-git-bot
Copy link
Copy Markdown
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@Reyes4711-S73
Copy link
Copy Markdown
Contributor

@OCA/crm-sales-marketing-maintainers Please, can you review/merge this PR?

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.