Raf Ven

Results 79 comments of Raf Ven

@bodedra I have made the changes + added some extra tests for a better code coverage.

@ventor-dev Some approvals are required for the MR to be merged!

I rebased this branch already with the main 13.0 branch, but it doesn't fix the issue. @fredzamoabg I don't know what is wrong with Travis. It seems a problem with...

> I can't say, as the configuration is the same than other repos: > > https://github.com/OCA/account-invoicing/blob/13.0/.travis.yml#L20 https://github.com/OCA/l10n-spain/blob/13.0/.travis.yml#L22 > > Have you run pre-commit locally? Yes i did, and it runs...

This module depends on https://github.com/OCA/account-invoicing/pull/1061. test-requirements.txt file added in separated commit.

Working on migration of account_coa_menu, account_tag_menu, account_type_menu, account_group into module account_menu https://github.com/OCA/account-financial-tools/pull/889

@pedrobaeza Can you remove the account_***_menu modules from the migration list? Maybe pointing to the account_menu module in the list?

> @BertVGroenendael can you please fix build? @ajaniszewska-dev This build cannot be fixed at this moment because it depends on account_invoice_base_invoicing_mode which is also in PR https://github.com/OCA/account-invoicing/pull/1061

> test-requirements.txt Do you mean this https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#185oca_dependenciestxt? Does this also work for modules in the same repository?

This module depends on https://github.com/OCA/account-invoicing/pull/1061. test-requirements.txt file added in separated commit.