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

[14.0] [MIG] assets_management #3078

Merged
merged 13 commits into from
Jan 5, 2023
Merged

[14.0] [MIG] assets_management #3078

merged 13 commits into from
Jan 5, 2023

Conversation

sergiocorato
Copy link
Contributor

@sergiocorato sergiocorato commented Dec 2, 2022 •
edited
Loading

Sostituisce #2704 recuperando le modifiche fatte pertinenti alla migrazione.
Miglioramenti di funzionalità saranno inseriti in PR separate.

@sergiocorato sergiocorato force-pushed the 14.0-mig-assets_management branch 4 times, most recently from ffc7e55 to 47babbf Compare December 2, 2022 15:37
@TheMule71 TheMule71 changed the title [MIG] assets_management [14.0] [MIG] assets_management Dec 2, 2022
@sergiocorato sergiocorato mentioned this pull request Dec 3, 2022
Closed
@sergiocorato sergiocorato force-pushed the 14.0-mig-assets_management branch 6 times, most recently from 0d7dfab to e283826 Compare December 5, 2022 06:29
@francesco-ooops
Copy link
Contributor

@sergiocorato ottimo, facci sapere quando è pronta per review funzionale

@sergiocorato
Copy link
Contributor Author

@sergiocorato ottimo, facci sapere quando è pronta per review funzionale

è già possibile farne, gli errori sono dovuti ai test che non sono ancora implementati correttamente

@sergiocorato sergiocorato force-pushed the 14.0-mig-assets_management branch 4 times, most recently from 0066cbc to dd79d28 Compare December 9, 2022 14:30
stefano-ooops
stefano-ooops reviewed Dec 14, 2022
View reviewed changes
Copy link

@stefano-ooops stefano-ooops left a comment

Choose a reason for hiding this comment

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

Testato in runboat:

  • caricamento manuale del cespite OK
  • calcolo ammortamento annuale OK
  • Caricamento cespite da fattura di acquisto ERRORE (allego video)
  • vendita cespite manuale (da scheda cespite) non calcola minus o plus
  • vendita cespite da fattura vendita ERRORE

@sergiocorato
Copy link
Contributor Author

Testato in runboat:

* caricamento manuale del cespite OK

* calcolo ammortamento annuale OK

* Caricamento cespite da fattura di acquisto ERRORE (allego video)

* vendita cespite manuale (da scheda cespite) non calcola minus o plus

* vendita cespite da fattura vendita ERRORE [
  ](https://recordit.co/cwyobfX0ob)

Grazie, attendo video/screen errori.

@stefano-ooops
Copy link

Testato in runboat:

* caricamento manuale del cespite OK

* calcolo ammortamento annuale OK

* Caricamento cespite da fattura di acquisto ERRORE (allego video)

* vendita cespite manuale (da scheda cespite) non calcola minus o plus

* vendita cespite da fattura vendita ERRORE

Grazie, attendo video/screen errori.

di seguito video caricamento cespite da fattura di acquisto.

https://recordit.co/cwyobfX0ob

@sergiocorato sergiocorato force-pushed the 14.0-mig-assets_management branch 3 times, most recently from 801a42f to 27764e1 Compare December 15, 2022 18:32
@sergiocorato
Copy link
Contributor Author

@stefano-ooops il punto 3 e 5 dovrebbero essere risolti con l'ultima modifica

@sergiocorato sergiocorato force-pushed the 14.0-mig-assets_management branch 3 times, most recently from 0b15caf to e2cfcce Compare December 20, 2022 11:40
stefano-ooops
stefano-ooops approved these changes Dec 23, 2022
View reviewed changes
Copy link

@stefano-ooops stefano-ooops left a comment

Choose a reason for hiding this comment

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

Testato funzionalità acquisto e vendita cespite da fattura (punto 3 e 5) funziona correttamente e calcola minusvalenza o plusvalenza in modo corretto

@francesco-ooops
Copy link
Contributor

@TheMule71 riusciresti a fare review? grazie!

@francesco-ooops
Copy link
Contributor

Ciao Sergio, ho fatto vari test e tranne queste due cose mi sembra tutto ok

@sergiocorato queste due si possono già testare?

@sergiocorato
Copy link
Contributor Author

Ciao Sergio, ho fatto vari test e tranne queste due cose mi sembra tutto ok

@sergiocorato queste due si possono già testare?

Solo la prima

@francesco-ooops
Copy link
Contributor

Ciao Sergio, ho fatto vari test e tranne queste due cose mi sembra tutto ok

@sergiocorato queste due si possono già testare?

Solo la prima

Grazie, mi sembra che il secondo comportamento fosse già presente in v12 per cui si può correggere in una successiva PR se non è un errore bloccante

@As400it si può approvare per te?

@sergiocorato
Copy link
Contributor Author

Ciao Sergio, ho fatto vari test e tranne queste due cose mi sembra tutto ok

image image

In relazione alla seconda segnalazione, ho aggiunto una colonna per mostrare che è corretto così, sono lo scarico del capitale e della minusvalenza per ogni tipologia (civile e fiscale) quindi 4 righe:

Screenshot(157)

@sergiocorato sergiocorato force-pushed the 14.0-mig-assets_management branch 2 times, most recently from fbed1d7 to dd5e774 Compare January 5, 2023 10:05
@GianmariaBucceri
Copy link

Ciao Sergio, ho eseguito il seguente test:

  • Acquisto bene e creazione cespite da fattura.
  • Generazione ammortamento
  • Vendita bene e dismissione tramite fattura
    Le registrazioni contabili generate sono corrette

@sergiocorato sergiocorato force-pushed the 14.0-mig-assets_management branch 2 times, most recently from d95d208 to 672c785 Compare January 5, 2023 10:39
@sergiocorato
Copy link
Contributor Author

Sto ripulendo i vari commenti e verificando le ultime note, spero in giornata di caricare la versione definitiva.

@sergiocorato
Copy link
Contributor Author

Ho caricato le ultime modifiche per supportare le fatture d'acquisto con IVA parzialmente deducibile, dopo il completamentto dei test per me è definitiva.

@sergiocorato
Copy link
Contributor Author

@elvise @stefano-ooops @As400it @GianmariaBucceri se volete fare una review finale e nel caso usare il bottone Approva, per me la PR è completa

MarcoCalcagni
MarcoCalcagni approved these changes Jan 5, 2023
View reviewed changes
Copy link
Contributor

@MarcoCalcagni MarcoCalcagni left a comment

Choose a reason for hiding this comment

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

testato in produzione ns. db

Approve Grazie di cuore

Grande Sergio

@sergiocorato
Copy link
Contributor Author

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 14.0-ocabot-merge-pr-3078-by-sergiocorato-bump-nobump, awaiting test results.

@francesco-ooops
Copy link
Contributor

Grande!!!

@matteoopenf
Copy link
Contributor

Grande

@OCA-git-bot OCA-git-bot merged commit bb88280 into OCA:14.0 Jan 5, 2023
@OCA-git-bot
Copy link
Contributor

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

@sergiocorato sergiocorato deleted the 14.0-mig-assets_management branch January 5, 2023 23:10
@francesco-ooops francesco-ooops mentioned this pull request Feb 3, 2023
Merged
@tafaRU tafaRU mentioned this pull request May 24, 2023
Open
76 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stefano-ooops stefano-ooops stefano-ooops approved these changes

@MarcoCalcagni MarcoCalcagni MarcoCalcagni approved these changes

Assignees
No one assigned
Labels
approved merged 🎉 migration
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.