Skip to content

[MIG] l10n_it_location_nuts: Migration to 16.0#4458

Merged
OCA-git-bot merged 16 commits into
OCA:16.0from
PyTech-SRL:16.0-mig-l10n_it_location_nuts
Apr 10, 2025
Merged

[MIG] l10n_it_location_nuts: Migration to 16.0#4458
OCA-git-bot merged 16 commits into
OCA:16.0from
PyTech-SRL:16.0-mig-l10n_it_location_nuts

Conversation

@renda-dev

@renda-dev renda-dev commented Nov 25, 2024 •
edited
Loading

Copy link
Copy Markdown
Contributor

Supersedes #3029

Migrazione di l10n_it_location_nuts alla versione 16.0

Modifiche particolari ai test per essere concordi con le linee guida OCA e risolve #3366

@renda-dev renda-dev force-pushed the 16.0-mig-l10n_it_location_nuts branch 2 times, most recently from 1a14620 to 3e026cd Compare November 25, 2024 12:55
@francesco-ooops francesco-ooops mentioned this pull request Nov 25, 2024
Open
3 tasks
@francesco-ooops francesco-ooops linked an issue Nov 25, 2024 that may be closed by this pull request
Aggiornamento API NUTS #3366
Open
3 tasks
HekkiMelody
HekkiMelody approved these changes Nov 26, 2024
View reviewed changes

@HekkiMelody HekkiMelody left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code review, LGTM

@SirAionTech

SirAionTech commented Nov 26, 2024

Copy link
Copy Markdown
Contributor

/ocabot migration l10n_it_location_nuts

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Nov 26, 2024
@OCA-git-bot OCA-git-bot mentioned this pull request Nov 26, 2024
Open
81 tasks
@github-actions

github-actions Bot commented Mar 30, 2025

Copy link
Copy Markdown

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions Bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Mar 30, 2025
SimoRubi and others added 16 commits April 1, 2025 10:35
@SimoRubi @PicchiSeba
[ADD] l10n_it_location_nuts (OCA#582)
81d7c5b 
* [ADD] l10n_it_location_nuts

* fixup! [ADD] l10n_it_location_nuts

* [FIX] align RAMON and Odoo Data

* [ADD] tests
@primes2h @PicchiSeba
Added translation using Weblate (Italian)
9fb6210
@sergiocorato @PicchiSeba
[12.0][ADD] it location nuts
48d7a49 
[FIX] readme

[FIX] terms

[FIX] readme

[FIX] configure

[FIX] configure
@sergiocorato @PicchiSeba
[FIX] missing sud sardegna
156b4d1
@primes2h @PicchiSeba
Translated using Weblate (Italian)
ac12b24 
Currently translated at 100.0% (1 of 1 strings)

Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_location_nuts
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_location_nuts/it/
@PicchiSeba
merge PR 1264 1717 1728 1737 1739
dea18de
@GSLabIt @PicchiSeba
[IMP] l10n_it_location_nuts: black, isort, prettier
2458f15
@GSLabIt @PicchiSeba
[14.0][MIG] l10n_it_location_nuts
c3214b0
@SirTakobi @PicchiSeba
[MIG] l10n_it_location_nuts: In tests, allow to store Odoo fields in …
b99c735 
…wizard

Otherwise `current_country_id` is not filled.
This wasn't needed in `12.0` because the wizard had no Odoo fields
@primes2h @PicchiSeba
[FIX] category in manifest file
74514f0
@francesco-ooops @PicchiSeba
Translated using Weblate (Italian)
b8f9f00 
Currently translated at 100.0% (4 of 4 strings)

Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_location_nuts
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_location_nuts/it/
@PicchiSeba
[FIX] Test commented to skip API call for NUTS list downloading OCA#3366
8df7beb
@francesco-ooops @PicchiSeba
Translated using Weblate (Italian)
4b91b03 
Currently translated at 100.0% (4 of 4 strings)

Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_location_nuts
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_location_nuts/it/
@renda-dev @PicchiSeba
[IMP] l10n_it_location_nuts: pre-commit execution
bf8fc5b
@renda-dev @PicchiSeba
[MIG] l10n_it_location_nuts: Migration to 16.0
7e97fa2
@renda-dev @PicchiSeba
[IMP] l10n_it_location_nuts: remove flaky tests
cf4fdc7
@PicchiSeba PicchiSeba force-pushed the 16.0-mig-l10n_it_location_nuts branch from 3e026cd to cf4fdc7 Compare April 1, 2025 08:36
@github-actions github-actions Bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Apr 6, 2025
Wvven
Wvven approved these changes Apr 10, 2025
View reviewed changes

@Wvven Wvven left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@OCA-git-bot

OCA-git-bot commented Apr 10, 2025

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). 🤖

MaurizioPellegrinet
MaurizioPellegrinet approved these changes Apr 10, 2025
View reviewed changes

@MaurizioPellegrinet MaurizioPellegrinet left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Test funzionale: OK

@tafaRU

tafaRU commented Apr 10, 2025

Copy link
Copy Markdown
Member

/ocabot merge nobump

@OCA-git-bot

OCA-git-bot commented Apr 10, 2025

Copy link
Copy Markdown
Contributor

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-4458-by-tafaRU-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 9918797 into OCA:16.0 Apr 10, 2025
@OCA-git-bot

OCA-git-bot commented Apr 10, 2025

Copy link
Copy Markdown
Contributor

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

@HekkiMelody HekkiMelody deleted the 16.0-mig-l10n_it_location_nuts branch June 12, 2025 09:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

3 more reviewers

@HekkiMelody HekkiMelody HekkiMelody approved these changes

@MaurizioPellegrinet MaurizioPellegrinet MaurizioPellegrinet approved these changes

@Wvven Wvven Wvven approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approved merged 🎉 migration ready to merge

Projects

None yet

Milestone

16.0

Development

Successfully merging this pull request may close these issues.

Aggiornamento API NUTS
⚡