Skip to content

[MIG] l10n_it_financial_statement_eu: Migration to 18.0 #4713

Merged
OCA-git-bot merged 9 commits into
OCA:18.0from
TheMule71:pr4559
Jun 6, 2025
Merged

[MIG] l10n_it_financial_statement_eu: Migration to 18.0 #4713
OCA-git-bot merged 9 commits into
OCA:18.0from
TheMule71:pr4559

Conversation

@TheMule71

@TheMule71 TheMule71 commented Apr 10, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

Superseeds #4559

Riorganizzati i commit. Adattata ai campi in l10n_it_edi

@OCA-git-bot

OCA-git-bot commented Apr 10, 2025

Copy link
Copy Markdown
Contributor

Hi @renda-dev, @aleuffre, @MarcoCalcagni,
some modules you are maintaining are being modified, check this out!

@TheMule71 TheMule71 marked this pull request as ready for review April 10, 2025 11:09
@TheMule71 TheMule71 mentioned this pull request Apr 10, 2025
Closed
@TheMule71 TheMule71 force-pushed the pr4559 branch 3 times, most recently from efc1356 to 2a70450 Compare April 10, 2025 12:50
@TheMule71

TheMule71 commented Apr 10, 2025

Copy link
Copy Markdown
Contributor Author

/ocabot migration l10n_it_financial_statement_eu

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Apr 10, 2025
@OCA-git-bot

OCA-git-bot commented Apr 10, 2025

Copy link
Copy Markdown
Contributor

The migration issue (#4392) has not been updated to reference the current pull request because a previous pull request (#4559) is not closed.
Perhaps you should check that there is no duplicate work.
CC @eLBati

@TheMule71 TheMule71 mentioned this pull request Apr 10, 2025
Open
46 tasks
@francesco-ooops

francesco-ooops commented Apr 10, 2025

Copy link
Copy Markdown
Contributor

@TheMule71 teniamo questa o #4559 ?

@TheMule71

TheMule71 commented Apr 11, 2025

Copy link
Copy Markdown
Contributor Author

@TheMule71 teniamo questa o #4559 ?

Questa. Ho già aggiornato la issue di migrazione. Ne abbiamo parlato in call con @eLBati

sergiocorato
sergiocorato reviewed Apr 11, 2025
View reviewed changes
Comment thread l10n_it_financial_statement_eu/__manifest__.py Outdated
"name": "ITA - Bilancio UE con XBRL",
"version": "18.0.1.0.0",
"category": "Localization/Italy",
"development_status": "Alpha",

@sergiocorato sergiocorato Apr 11, 2025

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.

Lo toglierei o metterei "Beta"

@TheMule71 TheMule71 May 16, 2025

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

fatto

Borruso added a commit to DinamicheAziendali/l10n-italy that referenced this pull request Apr 18, 2025
@Borruso
[UPD] l10n_it_financial_statement_eu: aggiornato PR OCA#4713
1f674b5
TennyMkt and others added 9 commits May 16, 2025 10:04
@TennyMkt @TheMule71
[MIG] l10n_it_financial_statement_eu: Migration to 16.0
a808adc 
migration script

[FIX] l10n_it_financial_statement: tolta _get_xlsx_name e altre migliorie
@SirAionTech @TheMule71
[FIX] l10n_it_financial_statement_eu: Ordering field must be in view
5c0cb5c 
Otherwise, when opening a line from the list view:
OwlError: An error occured in the owl lifecycle (see this Error's "cause" property)
    at handleError (http://oca-l10n-italy-16-0-pr3599-6f592ebc11da.runboat.odoo-community.org/web/assets/444-826ea0b/web.assets_common.min.js:1472:101)
    at owl.App.handleError (http://oca-l10n-italy-16-0-pr3599-6f592ebc11da.runboat.odoo-community.org/web/assets/444-826ea0b/web.assets_common.min.js:2100:29)
    at ComponentNode.initiateRender (http://oca-l10n-italy-16-0-pr3599-6f592ebc11da.runboat.odoo-community.org/web/assets/444-826ea0b/web.assets_common.min.js:1562:19)

Caused by: TypeError: Cannot read properties of undefined (reading 'type')
    at compareRecords (http://oca-l10n-italy-16-0-pr3599-6f592ebc11da.runboat.odoo-community.org/web/assets/445-43c5a33/web.assets_backend.min.js:8267:246)
    at stableCompare (http://oca-l10n-italy-16-0-pr3599-6f592ebc11da.runboat.odoo-community.org/web/assets/444-826ea0b/web.assets_common.min.js:6091:145)
    at Array.sort (<anonymous>)
    at Object.stableSort (http://oca-l10n-italy-16-0-pr3599-6f592ebc11da.runboat.odoo-community.org/web/assets/444-826ea0b/web.assets_common.min.js:6091:102)
    at Class._sortList (http://oca-l10n-italy-16-0-pr3599-6f592ebc11da.runboat.odoo-community.org/web/assets/445-43c5a33/web.assets_backend.min.js:8270:54)
    at http://oca-l10n-italy-16-0-pr3599-6f592ebc11da.runboat.odoo-community.org/web/assets/445-43c5a33/web.assets_backend.min.js:8237:113
    at async Promise.all (index 0)
    at async Promise.all (index 0)
    at async Class.load (http://oca-l10n-italy-16-0-pr3599-6f592ebc11da.runboat.odoo-community.org/web/assets/445-43c5a33/web.assets_backend.min.js:7927:244)
    at async Record.load (http://oca-l10n-italy-16-0-pr3599-6f592ebc11da.runboat.odoo-community.org/web/assets/445-43c5a33/web.assets_backend.min.js:2347:77)
@mymage @TheMule71
Translated using Weblate (Italian)
86816a3 
Currently translated at 100.0% (98 of 98 strings)

Translation: l10n-italy-16.0/l10n-italy-16.0-l10n_it_financial_statement_eu
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-16-0/l10n-italy-16-0-l10n_it_financial_statement_eu/it/
@SirAionTech @TheMule71
[REF] l10n_it_financial_statement_eu: Regenerate README
395d23d 
Fix OCA/oca-github-bot#297
@OCA-git-bot @TheMule71
[BOT] post-merge updates
9bd4cf9
@mymage @TheMule71
Translated using Weblate (Italian)
b5f81b4 
Currently translated at 100.0% (98 of 98 strings)

Translation: l10n-italy-16.0/l10n-italy-16.0-l10n_it_financial_statement_eu
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-16-0/l10n-italy-16-0-l10n_it_financial_statement_eu/it/
@TheMule71
[IMP] l10n_it_financial_statement_eu: pre-commit stuff
c06ff11
@dariodelzozzo @TheMule71
[MIG] l10n_it_financial_statement_eu: Migration to 18.0
9bddef6
@TheMule71
[IMP] l10n_it_financial_statement_eu: all companies are updated, stat…
3b03174 
…us -> BETA
gdemarco2esseti
gdemarco2esseti reviewed May 16, 2025
View reviewed changes

@gdemarco2esseti gdemarco2esseti 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.

Esito test funzionale: da fixare

In caso di conti non mappati per il bilancio attualmente indica solo che il bilancio non è quadrato.

Screenshot 2025-05-16 alle 10 26 43

Nelle versioni precedenti indicava i conti che necessitano di una mappatura

Screenshot 2025-05-16 alle 10 27 45

@TheMule71

TheMule71 commented May 16, 2025

Copy link
Copy Markdown
Contributor Author

Esito test funzionale: da fixare

In caso di conti non mappati per il bilancio attualmente indica solo che il bilancio non è quadrato.

Screenshot 2025-05-16 alle 10 26 43

Nelle versioni precedenti indicava i conti che necessitano di una mappatura

Screenshot 2025-05-16 alle 10 27 45

siamo sicuri il primo errore sia dovuto a conti non mappati?

Se io lo provo localmente ottengo:

image

@eLBati

eLBati commented Jun 6, 2025

Copy link
Copy Markdown
Member

In caso di conti non mappati per il bilancio attualmente indica solo che il bilancio non è quadrato

Nel senso che hai dei conti non mappati e la squadratura dipende da quelli?
Oppure il bilancio è non quadrqato per altri motivi?

MaurizioPellegrinet
MaurizioPellegrinet approved these changes Jun 6, 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
Lo sbilancio era dovuto a registrazioni errate

@TheMule71

TheMule71 commented Jun 6, 2025

Copy link
Copy Markdown
Contributor Author

/ocabot merge nobump

@OCA-git-bot

OCA-git-bot commented Jun 6, 2025

Copy link
Copy Markdown
Contributor

On my way to merge this fine PR!
Prepared branch 18.0-ocabot-merge-pr-4713-by-TheMule71-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 848efda into OCA:18.0 Jun 6, 2025
5 of 7 checks passed
@OCA-git-bot

OCA-git-bot commented Jun 6, 2025

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at 2de18b1. 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

Reviewers

@sergiocorato sergiocorato sergiocorato approved these changes

+2 more reviewers

@gdemarco2esseti gdemarco2esseti gdemarco2esseti left review comments

@MaurizioPellegrinet MaurizioPellegrinet MaurizioPellegrinet approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

18.0 approved merged 🎉 migration needs functional review

Projects

None yet

Milestone

18.0

Development

Successfully merging this pull request may close these issues.
⚡