Skip to content

fix: skip unresolved batch transactions and retry on next sync#332

Merged
barredterra merged 5 commits intoversion-15-hotfixfrom
bump-fintech-4
Feb 21, 2026
Merged

fix: skip unresolved batch transactions and retry on next sync#332
barredterra merged 5 commits intoversion-15-hotfixfrom
bump-fintech-4

Conversation

@barredterra
Copy link
Member

@barredterra barredterra commented Jan 21, 2026

  • Bump fintech to 7.9.0

    • Adds support for python 3.14
    • Fixes parsing of unresolved batch transactions
  • Skip unresolved batch transactions and retry on next sync

    When split_batch_transactions is enabled but camt.054 data is not yet available, the sync now marks the EBICS Request as "Skipped" instead of "Successful". This allows the scheduled job to retry the download on the next sync cycle once the bank provides the batch details.

    Changes:

    • Add an UnresolvedBatchTransactionError for transient batch resolution failures
    • Add "Skipped" status option to EBICS Request DocType
    • Return request log from execute_ebics_download for status updates
    • Mark request as "Skipped" when batch cannot be resolved
    • Log warning instead of error for this transient condition
    • Make split_batch_transactions=False explicit in upload_camt_file

When split_batch_transactions is enabled but camt.054 data is not yet
available, the sync now marks the EBICS Request as "Skipped" instead of
"Successful". This allows the scheduled job to retry the download on the
next sync cycle once the bank provides the batch details.

Changes:
- Add UnresolvedBatchTransactionError for transient batch resolution failures
- Add "Skipped" status option to EBICS Request DocType
- Return request log from execute_ebics_download for status updates
- Mark request as "Skipped" when batch cannot be resolved
- Log warning instead of error for this transient condition
- Make split_batch_transactions=False explicit in upload_camt_file
@barredterra barredterra changed the title bump fintech 4 fix: skip unresolved batch transactions and retry on next sync Jan 21, 2026
@barredterra barredterra requested a review from Copilot January 21, 2026 12:48
@barredterra
Copy link
Member Author

barredterra commented Jan 21, 2026

Tested user-triggered sync of C53 and C52, works as expected.

Todo:

  • Test with C54
  • Check this use case: the bank breaks down certain batch transactions in C53, but not all. Split Batch Transactions is enabled. C54 is not used.

This comment was marked as resolved.

@barredterra barredterra merged commit 6594ab3 into version-15-hotfix Feb 21, 2026
3 checks passed
@barredterra barredterra deleted the bump-fintech-4 branch February 21, 2026 22:44
@barredterra
Copy link
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.

2 participants