-
-
Notifications
You must be signed in to change notification settings - Fork 410
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
[13.0][ADD] account_bank_statement_import_oca_camt54: #246
[13.0][ADD] account_bank_statement_import_oca_camt54: #246
Conversation
8ee8615
to
c179bb9
Compare
extension logic for account_bank_statement_import_camt_oca
f91db70
to
520fde7
Compare
bed1ba5
to
d9cd8c2
Compare
d9cd8c2
to
41a531a
Compare
functional test ok |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just code review
def _complete_stmts_vals(self, stmts_vals, journal, account_number): | ||
"""Search partner from partner reference""" | ||
stmts_vals = super()._complete_stmts_vals(stmts_vals, journal, account_number) | ||
for st_vals in stmts_vals: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you can't do a mapped() ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't really how it can be optimized
stmts_vals will be returned as a result eventually and this is just dict not orm model
found_node = node.xpath(xpath_expr, namespaces={"ns": ns}) | ||
if found_node: | ||
name = found_node[0].text | ||
break |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you just comment this with functional explanation
This PR has the |
/ocabot merge |
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at 01c4592. Thanks a lot for contributing to OCA. ❤️ |
extension logic for account_bank_statement_import_camt_oca
UPD addon to create aditional line in statement account_bank_statement_import_transfer_move