[MIG] account_invoice_export: Migration to 14.0 #299
Conversation
|
|
There was a problem hiding this comment.
Is the sale module addition to the dependencies needed ?
There was a problem hiding this comment.
ah it was a mistake, fixed, thanks
There was a problem hiding this comment.
Should be deleted if not needed anymore
|
Also in the commit related to the pre-commit hooks "account_invoice_export: black, isort, prettier" are all changes generated by the hooks ? |
phuctranfxvn
force-pushed
the
14.0-mig-account_invoice_export
branch
from
571a1d6 to
a27b0d3
Compare
|
LG, there is just one more thing for me is this later commit 51b5aed#diff-8abaea68b746a4f70ee3b565778ece57657b089f5751cd9c41f66c8b5b18feb3 also need to be included. Thank you |
hailangvn
force-pushed
the
14.0-mig-account_invoice_export
branch
from
cde45af to
fd40320
Compare
|
@TDu, The commit has been included. Thanks for reviewing. |
Refactor to help integration by other module. An example is on l10n_switzerland/ebill_paynet.
This change the visibility of the send eBill button on the invoice. It is now only shown if the option is checked on the transmit method. Also improve the configuration on the transmit method adding information on the process.
hailangvn
force-pushed
the
14.0-mig-account_invoice_export
branch
from
fd40320 to
213e668
Compare
|
/ocabot merge minor |
|
What a great day to merge this nice PR. Let's do it! |
|
Congratulations, your PR was merged at a4ea509. Thanks a lot for contributing to OCA. ❤️ |
7 participants
Requires OCA modules:
account_invoice_transmit_methodwhich is being migrated in [14.0][MIG] account_invoice_transmit_method account-invoicing#823