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 Aggiunta possibilità di non stampare i cespiti venduti prima della data di stampa #3330

Merged
merged 1 commit into from
Aug 23, 2023
Merged

[FIX][14.0] assets_management Aggiunta possibilità di non stampare i cespiti venduti prima della data di stampa #3330

merged 1 commit into from
Aug 23, 2023

Conversation

sergiocorato
Copy link
Contributor

@sergiocorato sergiocorato commented May 25, 2023 •
edited
Loading

Aggiunta possibilità di non stampare i cespiti venduti nell'anno precedente alla data di stampa.

@sergiocorato sergiocorato marked this pull request as draft May 29, 2023 07:28
@sergiocorato sergiocorato force-pushed the 14.0-fix-assets_management-hide-sold-assets branch 2 times, most recently from f5676c3 to 5d1b7e9 Compare May 29, 2023 13:50
@sergiocorato sergiocorato marked this pull request as ready for review May 29, 2023 13:59
@sergiocorato sergiocorato changed the title [FIX][14.0] assets_management do not report assets sold before report date [FIX][14.0] assets_management Aggiunta possibilità di non stampare i cespiti venduti prima della data di stampa May 29, 2023
@sergiocorato sergiocorato force-pushed the 14.0-fix-assets_management-hide-sold-assets branch from 5d1b7e9 to a43bfbf Compare May 29, 2023 18:02
filipposaviori
filipposaviori suggested changes May 30, 2023
View reviewed changes
Copy link

@filipposaviori filipposaviori left a comment

Choose a reason for hiding this comment

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

il bottone "View Journal Report" non funziona

@sergiocorato sergiocorato force-pushed the 14.0-fix-assets_management-hide-sold-assets branch from a43bfbf to 47e6fde Compare May 30, 2023 12:49
@sergiocorato sergiocorato mentioned this pull request May 31, 2023
Closed
22 tasks
@Marianna-Marasco
Copy link

Nel libro cespiti i beni venduti sono nascosti, ma se aggiungo il flag "show sold assets" non li mostra.
Nel previsionale i beni venduti sono nascosti, se aggiungo il flag li mostra.
I beni distrutti/dismessi direttamente non sono nascosti in entrambi.
Come fare a distinguere la stampa con flag o senza? si potrebbe rendere visibile quando questo è selezionato?

@sergiocorato sergiocorato mentioned this pull request Jun 9, 2023
Closed
10 tasks
@sergiocorato sergiocorato force-pushed the 14.0-fix-assets_management-hide-sold-assets branch from 47e6fde to 4d0d1be Compare June 9, 2023 10:08
@sergiocorato sergiocorato force-pushed the 14.0-fix-assets_management-hide-sold-assets branch from 4d0d1be to b2aa573 Compare June 19, 2023 12:46
@Marianna-Marasco
Copy link

Confermo che in entrambi i libri il flag permette di visualizzare i beni dismessi/venduti

@sergiocorato
Copy link
Contributor Author

il bottone "View Journal Report" non funziona

Questa PR non risolve quell'errore, per cui è stata aperta un'altra PR: #3348

Ci sono diversi problemi sul modulo dei cespiti, per cui ho aperto diverse PR separate, in cui ognuna corregge un problema specifico. Si prega di testare solo la soluzione al problema indicato, grazie.

eLBati
eLBati reviewed Jul 6, 2023
View reviewed changes
assets_management/report/asset_journal.py
@@ -56,6 +56,7 @@ class Report(models.TransientModel):
show_totals = fields.Boolean()

show_category_totals = fields.Boolean()
show_sold_assets = fields.Boolean()
Copy link
Member

Choose a reason for hiding this comment

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

Quindi il comportmanto predefinito ora è di non mostrare i cespiti venduti, mentre prima era di mostrarli, giusto?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Esatto (venduti entro l'anno precedente a quello della stampa)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@eLBati posso ritenere chiusa questa osservazione?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ho aggiunto nel wizard di stampa un help che descrive questo comportamento.

odooNextev added a commit to odooNextev/l10n-italy that referenced this pull request Jul 7, 2023
@odooNextev
[IMP] Aggiunta PR OCA#3330 per cespiti già venduti
a1ff644
@sergiocorato
Copy link
Contributor Author

il bottone "View Journal Report" non funziona

Questa PR non risolve quell'errore, per cui è stata aperta un'altra PR: #3348

Ci sono diversi problemi sul modulo dei cespiti, per cui ho aperto diverse PR separate, in cui ognuna corregge un problema specifico. Si prega di testare solo la soluzione al problema indicato, grazie.

@filipposaviori puoi cortesemente aggiornare la revisione?

@sergiocorato sergiocorato force-pushed the 14.0-fix-assets_management-hide-sold-assets branch from b2aa573 to 00fccb1 Compare August 22, 2023 14:01
@sergiocorato sergiocorato force-pushed the 14.0-fix-assets_management-hide-sold-assets branch from 00fccb1 to 345f630 Compare August 22, 2023 14:07
@sergiocorato
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-3330-by-sergiocorato-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit b1c162f into OCA:14.0 Aug 23, 2023
@OCA-git-bot
Copy link
Contributor

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

odooNextev added a commit to odooNextev/l10n-italy that referenced this pull request Sep 11, 2023
@odooNextev
[IMP] Aggiunta PR OCA#3330 per cespiti già venduti
11a1b4b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eLBati eLBati eLBati left review comments

@aleuffre aleuffre aleuffre approved these changes

@filipposaviori filipposaviori filipposaviori approved these changes

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

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

Successfully merging this pull request may close these issues.
6 participants
@sergiocorato @Marianna-Marasco @OCA-git-bot @eLBati @aleuffre @filipposaviori