-
-
Notifications
You must be signed in to change notification settings - Fork 309
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
[14.0][FW] l10n_it_declaration_of_intent: FW of #2356 #3463
[14.0][FW] l10n_it_declaration_of_intent: FW of #2356 #3463
Conversation
…cords that exceeds the limit_amount of dichiarazione.intento.yearly.limit MIG v14: tweaked l10n_it_fatturapa_out_di as a result of the changes
0a7cd94
to
87bb24b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Da portare/verificare poi la #2412
/ocabot merge patch |
What a great day to merge this nice PR. Let's do it! |
It looks like something changed on |
It looks like something changed on |
Congratulations, your PR was merged at 4904f14. Thanks a lot for contributing to OCA. ❤️ |
@@ -253,6 +252,10 @@ def check_declarations_amounts(self, declarations): | |||
|
|||
def get_declaration_residual_amounts(self, declarations): | |||
"""Get residual amount for every `declarations`.""" | |||
plafond = self.env.user.company_id.declaration_yearly_limit_ids.filtered( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Per correttezza sarebbe self.company_id
Original PR #2356
Tracking issue: #2404
Permette di emettere dichiarazioni d'intento oltre il plafond teorico, aggiungendo un check su ciò che è stato davvero speso, e non solo ciò che è stato dichiarato.