Skip to content

feat: parse fees included in SEPATransaction#348

Merged
barredterra merged 1 commit intoversion-15-hotfixfrom
parse-included-fees
Feb 21, 2026
Merged

feat: parse fees included in SEPATransaction#348
barredterra merged 1 commit intoversion-15-hotfixfrom
parse-included-fees

Conversation

@barredterra
Copy link
Copy Markdown
Member

@barredterra barredterra commented Feb 19, 2026

Adds support for storing included fees from SEPA CAMT transaction details during EBICS import.

  • Uses Chrgs/TtlChrgsAndTaxAmt to populate included_fee on created Bank Transactions.
  • Adds robust amount parsing (handles comma decimals, empty values, and malformed inputs safely).
  • Skips included fee assignment when charge currency and transaction currency do not match.

Test Plan

  • Import a CAMT file where TtlChrgsAndTaxAmt/@Ccy matches transaction currency, then verify included_fee is populated.
  • Import a CAMT file with invalid/empty TtlChrgsAndTaxAmt and verify included_fee falls back to 0.0.
  • Import a CAMT file with mismatching currencies and verify included_fee is 0.0.

@barredterra barredterra merged commit 5917be8 into version-15-hotfix Feb 21, 2026
3 checks passed
@barredterra barredterra deleted the parse-included-fees branch February 21, 2026 16:32
@barredterra
Copy link
Copy Markdown
Member Author

🎉 This PR is included in version 15.32.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant