Skip to content

Conversation

@sergiocorato
Copy link

@sergiocorato sergiocorato commented Dec 31, 2025

Replace #1722 as it is closed.

patrickrwilson and others added 19 commits December 31, 2025 11:53
This module adds a priority field to contacts which can be configured within the configuration menu.

[UPD] LINT errors

fixed lint error, renamed test py file, added missing readme

[UPD] Removed unused file

removed incorrect py file

[FIX] wrong sequence val in test

Remove Test

[FIX] Data File Sequence

Fixed wrong sequence field name on data file.

[UPD] noupdate

Removed <data> element and moved noupdate into Odoo as suggested by reviewers.

[UPD] Reviewer Changes

Changes based on reviewer's suggestions

[UPD] Additional Review Changes

Additional changes requested from reviewers

[UPD] Sequence

Moved next sequence out of create method and into field def.

[UPD] Review Change

[FIX] LINT
Currently translated at 86.6% (13 of 15 strings)

Translation: partner-contact-14.0/partner-contact-14.0-partner_priority
Translate-URL: https://translation.odoo-community.org/projects/partner-contact-14-0/partner-contact-14-0-partner_priority/it/
Currently translated at 80.0% (12 of 15 strings)

Translation: partner-contact-14.0/partner-contact-14.0-partner_priority
Translate-URL: https://translation.odoo-community.org/projects/partner-contact-14-0/partner-contact-14-0-partner_priority/es/
Currently translated at 100.0% (15 of 15 strings)

Translation: partner-contact-14.0/partner-contact-14.0-partner_priority
Translate-URL: https://translation.odoo-community.org/projects/partner-contact-14-0/partner-contact-14-0-partner_priority/es/
Currently translated at 100.0% (15 of 15 strings)

Translation: partner-contact-14.0/partner-contact-14.0-partner_priority
Translate-URL: https://translation.odoo-community.org/projects/partner-contact-14-0/partner-contact-14-0-partner_priority/it/
@luisDIXMIT
Copy link

The PR title must be "[16.0] [MIG] partner_priority: Migration to 16.0" for easier comprehension.

Copy link

@luisDIXMIT luisDIXMIT left a comment

Choose a reason for hiding this comment

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

Code review and tested on runboat, LGTM!

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.

9 participants