Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[MIG] partner_identification: Migration to 15.0 #1436

Merged
merged 59 commits into from
Mar 31, 2023

Conversation

jdidderen-noviat
Copy link

@jdidderen-noviat jdidderen-noviat commented Dec 26, 2022

Since the PR #1270 seems to be (almost) abandoned, I backported the work done by @marielejeune for v16 in the PR #1397

@rousseldenis
Copy link
Contributor

/ocabot migration partner_identification

@rousseldenis
Copy link
Contributor

@jdidderen-noviat Thanks for this.

Could you follow migration guide ? https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-15.0

Especially the Full process part (inversing branches in git format-patch command) ?

You must keep commit authorship from @marielejeune ones

Copy link

@njeudy njeudy left a comment

Choose a reason for hiding this comment

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

code and test

@njeudy
Copy link

njeudy commented Jan 21, 2023

I think @marielejeune do the work in V16 and here we patch from v14 .. so .. he add marielejeune as contributor .. that ok for me I think :)

lmignon and others added 20 commits February 5, 2023 22:26
* [IMP] improve module description
* [IMP] Remove useless comments
* [FIX] Complete incomplete sentence
* [IMP] Replace field 'state' by 'status' in res_partner_id_number
* [IMP] Add new field 'Place of Issuance'
* [FIX] Readme formatting
* [IMP] status is now a selection field
* [IMP] use method to provide the default value for validation_code
* [IMP] Add help texts
* [FIX] Add missing constrains on category_id in res_partner.id_number
        The number must be validated also when we change the category
Allow for context override of validations using ``id_no_validate``
OCA#419)

* [IMP] partner_identification: Add field computation and inverses
* Add methods to allow for computation and inverse of an ID field of a specific category type

* [IMP] partner_identification: Add search option
[FIX] Tests

[FIX] data sequence

Fix openerp to odoo in comment

[REM] at_install, post_install

[ADD] Model descriptions

[REM] Extra spaces
Currently translated at 97.9% (46 of 47 strings)

Translation: partner-contact-12.0/partner-contact-12.0-partner_identification
Translate-URL: https://translation.odoo-community.org/projects/partner-contact-12-0/partner-contact-12-0-partner_identification/it/
Updated by Update PO files to match POT (msgmerge) hook in Weblate.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: partner-contact-12.0/partner-contact-12.0-partner_identification
Translate-URL: https://translation.odoo-community.org/projects/partner-contact-12-0/partner-contact-12-0-partner_identification/
Currently translated at 97.9% (46 of 47 strings)

Translation: partner-contact-12.0/partner-contact-12.0-partner_identification
Translate-URL: https://translation.odoo-community.org/projects/partner-contact-12-0/partner-contact-12-0-partner_identification/es/
@jdidderen-noviat jdidderen-noviat force-pushed the 15.0-mig-partner_identification branch from 6073506 to 7e447ab Compare February 5, 2023 21:35
@jdidderen-noviat
Copy link
Author

@rousseldenis Sorry for the delay. I was a bit silly when I did this PR. I did the format patch from 14.0 to 15.0 and then cherry pick the changes from @marielejeune instead of format patching from 16.0 to 15.0

@njeudy It was exactly that.

Thanks for the reviews :)

@OCA-git-bot
Copy link
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). 🤖

@NL66278
Copy link
Contributor

NL66278 commented Mar 6, 2023

@jdidderen-noviat Please squash some of the translation commits (for instance the consecutive ones for Croatian or Italian).

badbole and others added 2 commits March 6, 2023 19:00
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: partner-contact-16.0/partner-contact-16.0-partner_identification
Translate-URL: https://translation.odoo-community.org/projects/partner-contact-16-0/partner-contact-16-0-partner_identification/

Translated using Weblate (Italian)

Currently translated at 87.7% (43 of 49 strings)

Translation: partner-contact-16.0/partner-contact-16.0-partner_identification
Translate-URL: https://translation.odoo-community.org/projects/partner-contact-16-0/partner-contact-16-0-partner_identification/it/

Translated using Weblate (Italian)

Currently translated at 87.7% (43 of 49 strings)

Translation: partner-contact-16.0/partner-contact-16.0-partner_identification
Translate-URL: https://translation.odoo-community.org/projects/partner-contact-16-0/partner-contact-16-0-partner_identification/it/

Translated using Weblate (Croatian)

Currently translated at 30.6% (15 of 49 strings)

Translation: partner-contact-16.0/partner-contact-16.0-partner_identification
Translate-URL: https://translation.odoo-community.org/projects/partner-contact-16-0/partner-contact-16-0-partner_identification/hr/

Translated using Weblate (Croatian)

Currently translated at 51.0% (25 of 49 strings)

Translation: partner-contact-16.0/partner-contact-16.0-partner_identification
Translate-URL: https://translation.odoo-community.org/projects/partner-contact-16-0/partner-contact-16-0-partner_identification/hr/
@jdidderen-noviat jdidderen-noviat force-pushed the 15.0-mig-partner_identification branch from 7e447ab to 49bc193 Compare March 6, 2023 18:01
@jdidderen-noviat
Copy link
Author

@NL66278 Done

{
"name": "Partner Identification Numbers",
"category": "Customer Relationship Management",
"version": "15.0.1.0.1",
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"version": "15.0.1.0.1",
"version": "15.0.1.0.0",

Copy link
Author

Choose a reason for hiding this comment

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

@MiquelRForgeFlow The change is applied. Thanks for the review :)

@jdidderen-noviat jdidderen-noviat force-pushed the 15.0-mig-partner_identification branch from 49bc193 to 52a463b Compare March 28, 2023 12:21
@dreispt
Copy link
Member

dreispt commented Mar 31, 2023

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 15.0-ocabot-merge-pr-1436-by-dreispt-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Mar 31, 2023
Signed-off-by dreispt
@OCA-git-bot
Copy link
Contributor

It looks like something changed on 15.0 in the meantime.
Let me try again (no action is required from you).
Prepared branch 15.0-ocabot-merge-pr-1436-by-dreispt-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 101247a into OCA:15.0 Mar 31, 2023
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 6799d94. Thanks a lot for contributing to OCA. ❤️

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.