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

[14.0][MIG] account_financial_report: Migration to 14.0 #733

Closed
wants to merge 148 commits into from

Conversation

tupaq
Copy link
Contributor

@tupaq tupaq commented Oct 21, 2020

dependencies:

  • date_range

  • report_xlsx

@tupaq tupaq force-pushed the 14.0-mig-account_financial_report branch 3 times, most recently from fbd79c3 to 4259085 Compare October 21, 2020 23:40
@tupaq tupaq changed the title [MIG] account_financial_report: Migration to 14.0 [14.0][MIG] account_financial_report: Migration to 14.0 Oct 22, 2020
@tupaq tupaq force-pushed the 14.0-mig-account_financial_report branch 2 times, most recently from 5bd29e6 to 6cf6b7b Compare November 6, 2020 23:59
@JordiBForgeFlow
Copy link
Member

@tupaq Consider adding ForgeFlow/reporting-engine@b6fbc66 to fix the error in travis due to dependencies. When report_xlsx adjust the oca dependency as indicated in https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#oca-dependencies-txt

@tupaq tupaq force-pushed the 14.0-mig-account_financial_report branch 3 times, most recently from 8dc66c1 to d7c8c4f Compare November 7, 2020 12:24
@tupaq
Copy link
Contributor Author

tupaq commented Nov 7, 2020

@tupaq Consider adding ForgeFlow/reporting-engine@b6fbc66 to fix the error in travis due to dependencies. When report_xlsx adjust the oca dependency as indicated in https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#oca-dependencies-txt
@JordiBForgeFlow it's added and it doesn't work

@coleste coleste mentioned this pull request Nov 13, 2020
6 tasks
@pedrobaeza
Copy link
Member

Travis and runbot are red. Can you please check?

@coleste
Copy link

coleste commented Nov 18, 2020

And I don't know if server-ux should be in oca-dependencies-txt, since we depend on date_range.

@tupaq tupaq force-pushed the 14.0-mig-account_financial_report branch 2 times, most recently from 053304c to 1da4c66 Compare November 19, 2020 00:05
@tupaq
Copy link
Contributor Author

tupaq commented Nov 19, 2020

And I don't know if server-ux should be in oca-dependencies-txt, since we depend on date_range.

you are right. Patched

@tupaq tupaq force-pushed the 14.0-mig-account_financial_report branch from 1da4c66 to 9dd04a8 Compare November 26, 2020 01:15
JordiBForgeFlow and others added 10 commits November 25, 2020 21:52
…ort.

This commit includes move lines on the report date by default. These lines shouldn't be considered in the futur.
…with hierarchy.

Update indentation, remove empty lines from header.

Update test.

Update pylint.

Remove company_id on computing accounts, since account.group is not a company based model, filtering accounts is done on trial balance report.

Update account variables.

Improve condition in padding on accounts.

Add option to print hierarchy based on defined accounts/computed accounts.

Add VAT report, hierarchy from tax tags ans taxes.

Fix pylint, xlsx report generation header.

Update code to select code_prefix or name.

Update code to select code_prefix or name.

Update code to select code_prefix or name.

Fix domain in base amounts in vat report.

Change trial balance code_prefix or name.

Update trail balance, add tests for vat report.

Update pylint, amounts as monetary, many2one option on generation excels.

Update pulint.

Add VAT Report in readme.

Add VAT Report in readme.

Update array_agg.

Update array_agg.

Update array_agg.

Add option in VAT Report to be printed on Tax Tags - Tax Groups.

Add widget to hierarchy_on on trial balance.
Translated using Weblate (Arabic)

Currently translated at 95.7% (177 of 185 strings)

Translation: account-financial-reporting-11.0/account-financial-reporting-11.0-account_financial_report
Translate-URL: https://translation.odoo-community.org/projects/account-financial-reporting-11-0/account-financial-reporting-11-0-account_financial_report/ar/
ibuioli and others added 24 commits November 25, 2020 21:53
Currently translated at 100.0% (241 of 241 strings)

Translation: account-financial-reporting-13.0/account-financial-reporting-13.0-account_financial_report
Translate-URL: https://translation.odoo-community.org/projects/account-financial-reporting-13-0/account-financial-reporting-13-0-account_financial_report/es_AR/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: account-financial-reporting-13.0/account-financial-reporting-13.0-account_financial_report
Translate-URL: https://translation.odoo-community.org/projects/account-financial-reporting-13-0/account-financial-reporting-13-0-account_financial_report/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: account-financial-reporting-13.0/account-financial-reporting-13.0-account_financial_report
Translate-URL: https://translation.odoo-community.org/projects/account-financial-reporting-13-0/account-financial-reporting-13-0-account_financial_report/
Currently translated at 100.0% (242 of 242 strings)

Translation: account-financial-reporting-13.0/account-financial-reporting-13.0-account_financial_report
Translate-URL: https://translation.odoo-community.org/projects/account-financial-reporting-13-0/account-financial-reporting-13-0-account_financial_report/es_AR/
…gling

Depending on the calling method, data can be something different from a dictionary
(like `None` if no keyword argument is provided), so this will crash. Example:
mis_builder is crashing in its tests.

We prevent that checking for a falsy value before trying to access the dictionary.

We also avoid context mangling for non AFR reports (those without the key
`account_financial_report_lang` in the dictionary), not callign to `with_context`
in that case.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: account-financial-reporting-13.0/account-financial-reporting-13.0-account_financial_report
Translate-URL: https://translation.odoo-community.org/projects/account-financial-reporting-13-0/account-financial-reporting-13-0-account_financial_report/
Currently translated at 75.5% (185 of 245 strings)

Translation: account-financial-reporting-13.0/account-financial-reporting-13.0-account_financial_report
Translate-URL: https://translation.odoo-community.org/projects/account-financial-reporting-13-0/account-financial-reporting-13-0-account_financial_report/ro/
Currently translated at 78.0% (191 of 245 strings)

Translation: account-financial-reporting-13.0/account-financial-reporting-13.0-account_financial_report
Translate-URL: https://translation.odoo-community.org/projects/account-financial-reporting-13-0/account-financial-reporting-13-0-account_financial_report/ro/
Currently translated at 79.2% (194 of 245 strings)

Translation: account-financial-reporting-13.0/account-financial-reporting-13.0-account_financial_report
Translate-URL: https://translation.odoo-community.org/projects/account-financial-reporting-13-0/account-financial-reporting-13-0-account_financial_report/ro/
Currently translated at 100.0% (245 of 245 strings)

Translation: account-financial-reporting-13.0/account-financial-reporting-13.0-account_financial_report
Translate-URL: https://translation.odoo-community.org/projects/account-financial-reporting-13-0/account-financial-reporting-13-0-account_financial_report/es_AR/
@tupaq tupaq force-pushed the 14.0-mig-account_financial_report branch from 9dd04a8 to e154200 Compare November 26, 2020 02:54
@tupaq tupaq marked this pull request as ready for review November 30, 2020 01:52
@ventor-dev
Copy link

@tupaq , any news on this migration?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.