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

[IMP] account_statement_import_camt: multi account support #666

Open
wants to merge 1 commit into
base: 16.0
Choose a base branch
from
Open

[IMP] account_statement_import_camt: multi account support #666

wants to merge 1 commit into from

Commits on Feb 6, 2024

  1. [IMP] account_statement_import_camt: multi account support 

    The module account_statement_import_file supports handling files
containing informations related to multiple accounts, but this was not
taken advantage of by account_statement_import_camt.

We update the structure returned by parser.parse() to use the expected
format in case of multiple accounts encountered in CAMT file.

This has the side effect of fixing a bug in the previous implementation,
when all the transactions in a CAMT file would be parsed as being on the
same account (the one on the last statement found in the file), and
possibly even more wrong, in the currency of the last statement).
    @gurneyalex
    gurneyalex committed Feb 6, 2024
    Configuration menu
    Copy the full SHA
    63932d5 View commit details
    Browse the repository at this point in the history