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.

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

[FIX] Registrazione contabile per cespite crea sempre rettifica negativa #4018

Merged

Conversation

SirAionTech
Copy link
Contributor

Risolve #4017 per 14.0.

@francesco-ooops francesco-ooops linked an issue Mar 6, 2024 that may be closed by this pull request
2 tasks
@SirAionTech SirAionTech force-pushed the 14.0-fix-assets_management-negative_amend branch 3 times, most recently from 59516de to 8c26e98 Compare March 6, 2024 10:40
@SirAionTech SirAionTech marked this pull request as ready for review March 6, 2024 11:00
@SirAionTech SirAionTech force-pushed the 14.0-fix-assets_management-negative_amend branch from 8c26e98 to 8ddba69 Compare March 8, 2024 11:34
@SirAionTech
Copy link
Contributor Author

Fatto rebase per risolvere i conflitti

@SirAionTech SirAionTech force-pushed the 14.0-fix-assets_management-negative_amend branch from 8ddba69 to 4202928 Compare March 21, 2024 15:40
@SirAionTech
Copy link
Contributor Author

Fatto rebase per risolvere i conflitti

@MaurizioConte
Copy link

@SirAionTech: runboat non attivo per fare i test facciamo un rebase della PR ?

@francesco-ooops
Copy link
Contributor

/ocabot rebase

Deciding the depreciation type based on the move type does not work with entries because they could be either "in" or "out".
Everything depends on the sign of the account move lines amount: if it is positive then it is an "in" depreciation, otherwise "out".
@OCA-git-bot
Copy link
Contributor

Congratulations, PR rebased to 14.0.

@OCA-git-bot OCA-git-bot force-pushed the 14.0-fix-assets_management-negative_amend branch from 700dae8 to ca8e0fe Compare July 5, 2024 09:45
@francesco-ooops
Copy link
Contributor

@MaurizioConte pronta

@MaurizioConte
Copy link

@SirAionTech @francesco-ooops Test OK

Copy link

@MaurizioPellegrinet MaurizioPellegrinet left a comment

Choose a reason for hiding this comment

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

Test funzionale: OK

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

@SirAionTech
Copy link
Contributor Author

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 14.0-ocabot-merge-pr-4018-by-SirAionTech-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit baaf9fa into OCA:14.0 Jul 8, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at cebbc29. 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Registrazione contabile per cespite crea sempre rettifica negativa
5 participants