-
Notifications
You must be signed in to change notification settings - Fork 146
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
Migration of aeroo to v11 #103
Comments
great news! I'll definitely contribute to this asap. |
Great !!! I will also contribute to the migration |
@JuaniAizean Thanks! Every bit helps. |
@sraps thanks, I changed my username |
Hi @sraps @jjscarafia , I just tried to install the aeroo reports module for v11 and I had to make the following modifications to install it and it works: 1.- Delete parser_state and parser_def in ir_model.py in def _add_manual_models 2.- Change the action in report_view.xml I have the code in my repository |
I tested the module, the changes in the XML of @FreeDoo2018 are necessary. The parser_state issue is because parser_state is defined before it is created. Hence the module won't do a clean install. I get this error though: |
File "/odoo/customer/community/report_aeroo/models/report.py", line 343, in write |
Thanks @CasVissers My PR is available for revision |
I need to review the changes well, I see errors in the installation |
@CasVissers @sraps @jjscarafia I have corrected my PR, the changes in python were not necessary |
Great! I've answered on the PR (#114). Thanks! |
Lots of improvements ! :) New feedback in #115 |
Thanks @jjscarafia @CasVissers I will perform tests and I will give you my feedback |
For anyone following this repository, please consider contributing to this campaign to migrate aeroo reports to v11.
Perhaps as a contribution related to how aeroo has already helped or perhaps because you will need it on the future.
Thanks!
The text was updated successfully, but these errors were encountered: