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.

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

[15.0][MIG] account_statement_import_base #570

Merged

Conversation

MiquelRForgeFlow
Copy link
Contributor

Forward ported changes from #482.

The account_statement_import_file_reconciliation_widget is left to #566.

@OCA-git-bot
Copy link
Contributor

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

alexis-via and others added 6 commits February 23, 2023 16:09
The 2 modules account_statement_import_online and
account_statement_import depend on account_statement_import_base (and
not on each other) and share common code, in particular a hook to update
the statement line. So we can now have reconciliation modules that use
this hook and therefore work both on file import and online import. More
details on OCA#481.

Improve bank statement line form view and journal form view.
account_statement_import_file_reconciliation_widget is a glue
auto-install module between account_statement_import (will be renamed
account_statement_import_file in v16) and account_reconciliation_widget.
@MiquelRForgeFlow MiquelRForgeFlow force-pushed the 15.0-mig-account_statement_import_base branch from 5fa720c to ba9e9d9 Compare February 23, 2023 15:09
@MiquelRForgeFlow
Copy link
Contributor Author

MiquelRForgeFlow commented Feb 23, 2023

Could we merge this ASAP? It's to avoid more diversion between versions. cc: @pedrobaeza

Copy link

@AaronHForgeFlow AaronHForgeFlow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code review

Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 15.0-ocabot-merge-pr-570-by-pedrobaeza-bump-nobump, awaiting test results.

@pedrobaeza pedrobaeza added this to the 15.0 milestone Feb 23, 2023
@OCA-git-bot OCA-git-bot merged commit c671758 into OCA:15.0 Feb 23, 2023
@OCA-git-bot
Copy link
Contributor

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

@MiquelRForgeFlow MiquelRForgeFlow deleted the 15.0-mig-account_statement_import_base branch February 23, 2023 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants