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

[FIX][14.0] assets_management depreciable amount computation by date #3316

Merged
merged 2 commits into from
Jul 7, 2023
Merged

[FIX][14.0] assets_management depreciable amount computation by date #3316

merged 2 commits into from
Jul 7, 2023

Conversation

sergiocorato
Copy link
Contributor

@sergiocorato sergiocorato commented May 19, 2023 •
edited
Loading

risolve #3318

@dcorio
Copy link
Contributor

dcorio commented May 19, 2023

@sergiocorato c'è una issue collegata? giusto per capire cosa testare

@sergiocorato
Copy link
Contributor Author

@dcorio No, sto ancora finendo di creare i test, poi la creo.

@sergiocorato sergiocorato force-pushed the 14.0-fix-assets_management-depreciation-with-incremented-value branch from c86fab2 to b433e34 Compare May 19, 2023 15:34
@sergiocorato sergiocorato mentioned this pull request May 19, 2023
Closed
3 tasks
@sergiocorato
Copy link
Contributor Author

sergiocorato commented May 19, 2023 •
edited
Loading

Ho mantenuto in un commit separato la correzione del report previsionale di stampa dei cespiti.

@sergiocorato sergiocorato force-pushed the 14.0-fix-assets_management-depreciation-with-incremented-value branch 3 times, most recently from c36ea4e to c2b7e5a Compare May 23, 2023 14:03
@sergiocorato sergiocorato force-pushed the 14.0-fix-assets_management-depreciation-with-incremented-value branch from c2b7e5a to e613af7 Compare May 31, 2023 09:31
@sergiocorato sergiocorato changed the title [FIX][14.0] depreciable amount computation by date [FIX][14.0] assets_management depreciable amount computation by date May 31, 2023
@sergiocorato sergiocorato mentioned this pull request May 31, 2023
Closed
22 tasks
@sergiocorato sergiocorato mentioned this pull request Jun 9, 2023
Closed
10 tasks
@matteoopenf
Copy link
Contributor

@sergiocorato per caso la pr e' finita per fare review ecc?

@sergiocorato
Copy link
Contributor Author

@sergiocorato per caso la pr e' finita per fare review ecc?

@matteoopenf Sì, le PR inserite nella #3345 sono revisionabili (ho cercato di renderle autosufficienti per quanto possibile, quindi testabili separamente)

@matteoopenf
Copy link
Contributor

@sergiocorato per caso la pr e' finita per fare review ecc?

@matteoopenf Sì, le PR inserite nella #3345 sono revisionabili (ho cercato di renderle autosufficienti per quanto possibile, quindi testabili separamente)

Grazie Mille

matteoopenf
matteoopenf approved these changes Jun 28, 2023
View reviewed changes
Copy link
Contributor

@matteoopenf matteoopenf left a comment

Choose a reason for hiding this comment

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

testato funzionalmente

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

@sergiocorato
Copy link
Contributor Author

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot OCA-git-bot merged commit 87d789b into OCA:14.0 Jul 7, 2023
@OCA-git-bot
Copy link
Contributor

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

odooNextev added a commit to odooNextev/l10n-italy that referenced this pull request Sep 11, 2023
@odooNextev
[FIX] Aggiunta PR OCA#3316 per depreciable amount computation by date
3fabee7
@francesco-ooops francesco-ooops linked an issue Dec 21, 2023 that may be closed by this pull request
Calcolo di ammortamenti su cespiti completamente ammortizzati con incremento valore successivo #3318
Closed
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@matteoopenf matteoopenf matteoopenf approved these changes

@Marianna-Marasco Marianna-Marasco Marianna-Marasco approved these changes

Assignees
No one assigned
Labels
approved merged 🎉 ready to merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Calcolo di ammortamenti su cespiti completamente ammortizzati con incremento valore successivo
5 participants
@sergiocorato @dcorio @matteoopenf @OCA-git-bot @Marianna-Marasco