bank-statement-import icon indicating copy to clipboard operation
bank-statement-import copied to clipboard

[IMP] account_bank_statement_import_txt_xlsx: Import lines to existing bank statement

Open zaoral opened this issue 2 years ago • 1 comments

zaoral avatar Aug 18 '22 21:08 zaoral

Hi @alexey-pelykh, some modules you are maintaining are being modified, check this out!

OCA-git-bot avatar Aug 18 '22 21:08 OCA-git-bot

I don't think such feature should be implemented at the level of individual file parser. Take a look at https://github.com/OCA/bank-statement-import/tree/12.0/account_bank_statement_import_split maybe it already solves the case that you need.

alexey-pelykh avatar Nov 14 '22 21:11 alexey-pelykh

I agree with @alexey-pelykh that it would be best to add it to https://github.com/OCA/bank-statement-import/tree/13.0/account_bank_statement_import_split.

However in my opinion there's a conflict in: https://github.com/OCA/bank-statement-import/tree/13.0/account_bank_statement_import_split.

In my opinion the decision to group the statements to daily, weekly, monthly should be stored in the journal, similar to how you can now define it in online statement import https://github.com/OCA/bank-statement-import/blob/14.0/account_statement_import_online/models/online_bank_statement_provider.py#L78

I will send a PR.

JordiBForgeFlow avatar Dec 16 '22 10:12 JordiBForgeFlow

name with version please

andreampiovesana avatar Jul 03 '23 15:07 andreampiovesana

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

github-actions[bot] avatar Nov 05 '23 12:11 github-actions[bot]