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.

Sign up for GitHub

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

Jump to bottom

[MIG] assets_management -> l10n_it_asset_management: Migration to 16.0 #3776

Merged
merged 52 commits into from
Apr 12, 2024
Merged

[MIG] assets_management -> l10n_it_asset_management: Migration to 16.0 #3776

merged 52 commits into from
Apr 12, 2024

Conversation

SirAionTech
Copy link
Contributor

@SirAionTech SirAionTech commented Dec 12, 2023 •
edited
Loading

Migrazione da https://github.com/OCA/l10n-italy/tree/2f71528a4b249e2adf33d18108247c80f15f55b1/assets_management (14.0).
Sostituisce #3271.

Il commit successivo alla migrazione risolve #3775 per 16.0.
Risolve #3009 per 16.0.
Risolve #3318 per 16.0.
Risolve #3851 per 16.0.
Risolve #3874 per 16.0.
Risolve #3170 per 16.0.

@SirAionTech SirAionTech marked this pull request as ready for review December 12, 2023 16:06
@SirAionTech SirAionTech mentioned this pull request Dec 12, 2023
Closed
@filipposaviori
Copy link

Domani faccio test.

andreampiovesana
andreampiovesana suggested changes Dec 13, 2023
View reviewed changes
Copy link
Contributor

@andreampiovesana andreampiovesana left a comment

Choose a reason for hiding this comment

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

lo rinimini in l10n_it_asset_management?

@SirAionTech
Copy link
Contributor Author

lo rinimini in l10n_it_asset_management?

Ah grazie mi sono dimenticato di scriverlo: rispetto a #3271 non c'è più la rinomina del modulo.
Non l'ho aggiunta perché non ho trovato nessuna discussione o motivazione per aggiungerla.
Tu sai mica chi l'ha deciso e perché è stato deciso di rinominarlo?

@andreampiovesana
Copy link
Contributor

credo per sottolineare la specificità italiana del modulo, visto che la precedente PR faceva così manterrei la rinomina del modulo.

@francesco-ooops
Copy link
Contributor

@SirAionTech era stato discusso qua #3155 (comment) ed anche io lo ritengo corretto (anche solo per non buttare via il lavoro di @odooNextev )

@primes2h
Copy link
Contributor

@SirAionTech era stato discusso qua #3155 (comment)

Esatto, nello specifico la discussione era partita qui.

@SirAionTech SirAionTech marked this pull request as draft December 14, 2023 08:32
@SirAionTech SirAionTech changed the title [MIG] assets_management: Migration to 16.0 [MIG] assets_management -> l10n_it_asset_management: Migration to 16.0 Dec 14, 2023
@SirAionTech SirAionTech marked this pull request as ready for review December 14, 2023 11:36
@SirAionTech
Copy link
Contributor Author

Aggiunta rinomina del modulo, provata con https://github.com/OCA/OpenUpgrade

@francesco-ooops
Copy link
Contributor

@SirAionTech potrebbe essere la buona occasione di aggiungere una copertura di test?

@SirAionTech
Copy link
Contributor Author

@SirAionTech potrebbe essere la buona occasione di aggiungere una copertura di test?

Sì, se/quando ho tempo li aggiungo.

A occhio la mancanza grossa di copertura è sul codice dei report, quindi potrei partire dal porting di #3010.

@francesco-ooops francesco-ooops mentioned this pull request Dec 18, 2023
Open
79 tasks
This was linked to issues Dec 18, 2023
Migration to version 16.0 #2967
Open
Azioni client inutilizzate nel modulo dei cespiti #3775
Open
@francesco-ooops francesco-ooops mentioned this pull request Dec 18, 2023
Open
2 tasks
@SirAionTech SirAionTech marked this pull request as draft December 19, 2023 14:07
@SirAionTech SirAionTech force-pushed the 16.0-mig-assets_management branch 2 times, most recently from 1d58cae to 577dff4 Compare December 19, 2023 14:12
@SirAionTech SirAionTech mentioned this pull request Dec 19, 2023
Closed
3 tasks
@SirAionTech SirAionTech marked this pull request as ready for review December 19, 2023 14:28
@francesco-ooops francesco-ooops linked an issue Dec 19, 2023 that may be closed by this pull request
Il fondo ammortizzato nell'anno precedente non considera tutti gli anni precedenti #3009
Closed
3 tasks
@gdemarco2esseti
Copy link

Buon pomeriggio a tutti. Sono una collega di @filipposaviori. Ho notato che il runboat di test ha solo l'app di fatturazione senza le funzionalità di contabilità.
immagine

Non riesco ad associare una fattura di acquisto ad un cespite perchè non è possibile configurare le scritture contabili e i conti ad esse asasociate. Come posso risolvere?

Grazie mille.
Giulia

@SirAionTech
Copy link
Contributor Author

Buon pomeriggio a tutti. Sono una collega di @filipposaviori. Ho notato che il runboat di test ha solo l'app di fatturazione senza le funzionalità di contabilità. immagine

Non riesco ad associare una fattura di acquisto ad un cespite perchè non è possibile configurare le scritture contabili e i conti ad esse asasociate. Come posso risolvere?

Grazie mille. Giulia

Grazie di aver provato la PR!
Forse ti basta aggiungere il tuo utente al gruppo "Funzionalità contabili complete"?

@gdemarco2esseti
Copy link

Buon pomeriggio a tutti. Sono una collega di @filipposaviori. Ho notato che il runboat di test ha solo l'app di fatturazione senza le funzionalità di contabilità. immagine
Non riesco ad associare una fattura di acquisto ad un cespite perchè non è possibile configurare le scritture contabili e i conti ad esse asasociate. Come posso risolvere?
Grazie mille. Giulia

Grazie di aver provato la PR! Forse ti basta aggiungere il tuo utente al gruppo "Funzionalità contabili complete"?

Grazie per il celere feedback. Avevo già verificato. L'utente è configurato con "Responsabile fatturazione". Non è possibile assegnare le funzioni contabili complete

@SirAionTech
Copy link
Contributor Author

Buon pomeriggio a tutti. Sono una collega di @filipposaviori. Ho notato che il runboat di test ha solo l'app di fatturazione senza le funzionalità di contabilità. immagine
Non riesco ad associare una fattura di acquisto ad un cespite perchè non è possibile configurare le scritture contabili e i conti ad esse asasociate. Come posso risolvere?
Grazie mille. Giulia

Grazie di aver provato la PR! Forse ti basta aggiungere il tuo utente al gruppo "Funzionalità contabili complete"?

Grazie per il celere feedback. Avevo già verificato. L'utente è configurato con "Responsabile fatturazione". Non è possibile assegnare le funzioni contabili complete

In 16.0 non c'è l'opzione per poterlo fare dal form dell'utente, a quanto ne so l'unico modo è aprire il gruppo (i gruppi si vedono solo in modalità debug) e aggiungere il proprio utente al gruppo.

@gdemarco2esseti
Copy link

Buon pomeriggio a tutti. Sono una collega di @filipposaviori. Ho notato che il runboat di test ha solo l'app di fatturazione senza le funzionalità di contabilità. immagine
Non riesco ad associare una fattura di acquisto ad un cespite perchè non è possibile configurare le scritture contabili e i conti ad esse asasociate. Come posso risolvere?
Grazie mille. Giulia

Grazie di aver provato la PR! Forse ti basta aggiungere il tuo utente al gruppo "Funzionalità contabili complete"?

Grazie per il celere feedback. Avevo già verificato. L'utente è configurato con "Responsabile fatturazione". Non è possibile assegnare le funzioni contabili complete

In 16.0 non c'è l'opzione per poterlo fare dal form dell'utente, a quanto ne so l'unico modo è aprire il gruppo (i gruppi si vedono solo in modalità debug) e aggiungere il proprio utente al gruppo.

Perfetto. Ora ho accesso alle funzionalità contabili complete. Grazie mille per il feedback.

@SirAionTech SirAionTech mentioned this pull request Mar 6, 2024
Closed
2 tasks
@gdemarco2esseti
Copy link

Buon pomeriggio a tutti. Ho completato il test sul runboat senza riscontrare alcun problema. Di seguito un riepilogo delle funzionalità testate:

  • creazione di un cespite direttamente da una fattura
  • la creazione di un cespite dal modulo e il relativo collegamento con la fattura
  • scritture di ammortamento
  • collegamento con le scritture di svalutazione/rivalutazione
  • stampa del libro cespiti

Spero che questo feedback possa essere utile.

Un saluto,
Giulia

@SirAionTech
Copy link
Contributor Author

Buon pomeriggio a tutti. Ho completato il test sul runboat senza riscontrare alcun problema. Di seguito un riepilogo delle funzionalità testate:

  • creazione di un cespite direttamente da una fattura
  • la creazione di un cespite dal modulo e il relativo collegamento con la fattura
  • scritture di ammortamento
  • collegamento con le scritture di svalutazione/rivalutazione
  • stampa del libro cespiti

Spero che questo feedback possa essere utile.

Un saluto, Giulia

Grazie mille! Potresti approvare la PR?
Puoi vedere come in https://www.odoo-italia.org/documentazione/14.0/sviluppo/review.html#revisione

@SirAionTech
Copy link
Contributor Author

Inclusa anche #4015.

gdemarco2esseti
gdemarco2esseti approved these changes Mar 7, 2024
View reviewed changes
Copy link

@gdemarco2esseti gdemarco2esseti left a comment

Choose a reason for hiding this comment

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

Funzionalità testate:

  • creazione di un cespite direttamente da una fattura

  • la creazione di un cespite dal modulo e il relativo collegamento con la fattura

  • scritture di ammortamento

  • collegamento con le scritture di svalutazione/rivalutazione

  • stampa del libro cespiti

@SirAionTech SirAionTech mentioned this pull request Mar 8, 2024
Open
@SirAionTech SirAionTech mentioned this pull request Mar 25, 2024
Merged
@SirAionTech SirAionTech mentioned this pull request Mar 26, 2024
Merged
Esa199
Esa199 approved these changes Mar 27, 2024
View reviewed changes
Copy link

@Esa199 Esa199 left a comment

Choose a reason for hiding this comment

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

Il modulo funziona correttamente, di seguito alcuni test svolti:

creazione di un nuovo cespite dal modulo e il relativo collegamento con la fattura
creazione di un cespite direttamente da una fattura
scritture di ammortamento
verifica differenziazione ammortamento civile e fiscale
collegamento con le scritture di svalutazione/rivalutazione
stampa registro cespiti e previsionale

@SirAionTech SirAionTech mentioned this pull request Apr 10, 2024
Open
2 tasks
@francesco-ooops
Copy link
Contributor

@OCA/local-italy-maintainers mergiamo?

@eLBati
Copy link
Member

eLBati commented Apr 12, 2024

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot OCA-git-bot merged commit 90303d9 into OCA:16.0 Apr 12, 2024
5 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@francesco-ooops francesco-ooops mentioned this pull request Apr 18, 2024
Closed
3 tasks
@PicchiSeba PicchiSeba mentioned this pull request Jul 11, 2024
Open
@SirAionTech SirAionTech removed a link to an issue Jul 15, 2024
Migration to version 16.0 #2967
Open
79 tasks
@TheMule71 TheMule71 mentioned this pull request Nov 15, 2024
Open
32 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Esa199 Esa199 Esa199 approved these changes

@gdemarco2esseti gdemarco2esseti gdemarco2esseti approved these changes

@eLBati eLBati eLBati approved these changes

@andreampiovesana andreampiovesana Awaiting requested review from andreampiovesana

Assignees
No one assigned
Labels
16.0 merged 🎉 migration needs functional review
Projects
None yet
Milestone
No milestone