[14.0][IMP]account_statement_import_txt_xlsx: additional mapping controls

OCA / bank-statement-import

Bank Import Addons

https://odoo-community.org/psc-teams/banking-10

GNU Affero General Public License v3.0

152 stars 412 forks source link

[14.0][IMP]account_statement_import_txt_xlsx: additional mapping controls #688

Closed PicchiSeba closed 5 months ago

PicchiSeba commented 6 months ago

These changes allow to skip a specified amount of rows/columns during the file parsing phase. There is also an option that allows to skip completely empy lines.

Please let me know what you think about it and if you like these changes to be made differently.

OCA-git-bot commented 6 months ago

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

PicchiSeba commented 5 months ago

@alexey-pelykh I added a couple of test that cover the new features. Are they good enough or do we require more in-depth test scenarios?

alexey-pelykh commented 5 months ago

/ocabot merge minor

OCA-git-bot commented 5 months ago

On my way to merge this fine PR! Prepared branch 14.0-ocabot-merge-pr-688-by-alexey-pelykh-bump-minor, awaiting test results.

alexey-pelykh commented 5 months ago

Please take some time to back- and forward- port the improvement.

OCA-git-bot commented 5 months ago

Congratulations, your PR was merged at 25ec27bb16aac26d9e73acd8c1a4eb9944a9cf00. Thanks a lot for contributing to OCA. ❤️