Skip to content

Arredondamento de valores financeiros em SAFT#46

Merged
albertomandlate merged 1 commit intomainfrom
hotfix-limitar-2-casas-decimais
Jun 23, 2025
Merged

Arredondamento de valores financeiros em SAFT#46
albertomandlate merged 1 commit intomainfrom
hotfix-limitar-2-casas-decimais

Conversation

@albertomandlate
Copy link
Copy Markdown
Contributor

Melhora a precisão dos cálculos financeiros no arquivo MozambiqueSaftGenerator.cs ao adicionar arredondamento para duas casas decimais em campos como TaxPercentage, TaxAmount, TotalDebit, TotalCredit, UnitPrice, TaxBase, DebitAmount, CreditAmount, TaxPayable, NetTotal e GrossTotal.

Melhora a precisão dos cálculos financeiros no arquivo `MozambiqueSaftGenerator.cs` ao adicionar arredondamento para duas casas decimais em campos como `TaxPercentage`, `TaxAmount`, `TotalDebit`, `TotalCredit`, `UnitPrice`, `TaxBase`, `DebitAmount`, `CreditAmount`, `TaxPayable`, `NetTotal` e `GrossTotal`.
@albertomandlate albertomandlate requested a review from Copilot June 23, 2025 09:00
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR improves the precision of financial calculations by rounding various monetary values to two decimal places.

  • Rounds financial fields such as TaxPercentage, TaxAmount, TotalDebit, TotalCredit, UnitPrice, TaxBase, DebitAmount, CreditAmount, TaxPayable, NetTotal, and GrossTotal.
  • Applies Math.Round consistently to ensure accuracy in the SAFT generator output.

Comment thread src/SAF-T.Mozambique/Generators/MozambiqueSaftGenerator.cs
Comment thread src/SAF-T.Mozambique/Generators/MozambiqueSaftGenerator.cs
@albertomandlate
Copy link
Copy Markdown
Contributor Author

Está tudo em conformidade

@albertomandlate albertomandlate merged commit 85ef2ba into main Jun 23, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants