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][FIX] assets_management get in-out values and minor fixes #3350

Closed
wants to merge 1 commit into from
Closed

[14.0][FIX] assets_management get in-out values and minor fixes #3350

wants to merge 1 commit into from

Conversation

sergiocorato
Copy link
Contributor

No description provided.

@sergiocorato sergiocorato force-pushed the 14.0-fix-assets_management-get-in-out-values branch from 693ee9c to c38afbb Compare May 31, 2023 12:56
@sergiocorato sergiocorato changed the title [14.0][FIX] do not create depreciation lines with zero amount [14.0][FIX] assets_management get in-out values and minor fixes 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
aleuffre
aleuffre approved these changes Jul 6, 2023
View reviewed changes
Copy link
Contributor

@aleuffre aleuffre left a comment

Choose a reason for hiding this comment

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

Code review, LGTM

@sergiocorato sergiocorato force-pushed the 14.0-fix-assets_management-get-in-out-values branch from c38afbb to e1c924d Compare August 30, 2023 07:55
@sergiocorato
Copy link
Contributor Author

/ocabot rebase

@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-get-in-out-values branch from e1c924d to 715b89a Compare September 21, 2023 15:29
@francesco-ooops francesco-ooops linked an issue Dec 20, 2023 that may be closed by this pull request
Refactoring cespiti per correzioni vari bug (assets_management) #3351
Closed
22 tasks
@github-actions GitHub Actions
Copy link

github-actions bot commented May 5, 2024

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label May 5, 2024
@francesco-ooops
Copy link
Contributor

@sergiocorato questa cosa risolve?

@sergiocorato
Copy link
Contributor Author

@sergiocorato questa cosa risolve?

È l'ultimo punto rimasto aperto qui #3351

@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label May 12, 2024
@francesco-ooops
Copy link
Contributor

/ocabot rebase

@OCA-git-bot
Copy link
Contributor

@francesco-ooops The rebase process failed, because command git push --force efatto tmp-pr-3350:14.0-fix-assets_management-get-in-out-values failed with output:

remote: Permission to efatto/l10n-italy.git denied to OCA-git-bot.
fatal: unable to access 'https://github.com/efatto/l10n-italy/': The requested URL returned error: 403

@francesco-ooops
Copy link
Contributor

@sergiocorato cosa si può fare per testare funzionalmente?

@sergiocorato
Copy link
Contributor Author

Chiudo questa PR in quanto non so praticamente a cosa serva, nel caso la riaprirò in seguito.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aleuffre aleuffre aleuffre approved these changes

Assignees
No one assigned
Labels
14.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Refactoring cespiti per correzioni vari bug (assets_management)
4 participants
@sergiocorato @OCA-git-bot @francesco-ooops @aleuffre