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

[13.0][FIX] l10n_th_tax_report, error on install #66

Merged

Conversation

kittiu
Copy link
Member

@kittiu kittiu commented Jan 20, 2020

This module still depends on

Error on install,

  File "/home/odoo/src/odoo/odoo/fields.py", line 3151, in _setup_regular_full
    raise TypeError(msg % (self, field))
TypeError: Many2many fields tax.report.view.sale_line_ids and account.move.line.sale_line_ids use the same table and columns

Note: now I use oca_dependencies.txt to ttps://github.com/OCA/reporting-engine/pull/361
so, it can works now. But still, we need to merge this PR.

@kittiu kittiu force-pushed the 13.0-fix-l10n_th_tax_report-error_install branch from 1b64b4a to b3d863d Compare January 20, 2020 04:29
@OCA-git-bot OCA-git-bot mentioned this pull request Jan 20, 2020
8 tasks
@kittiu
Copy link
Member Author

kittiu commented Jan 28, 2020

Now, it should be ready, await for report_xlsx_helper to get merged.

@kittiu kittiu force-pushed the 13.0-fix-l10n_th_tax_report-error_install branch from b3d863d to 1a2781f Compare January 31, 2020 05:27
@kittiu kittiu merged commit af31abc into OCA:13.0 Jan 31, 2020
@Saran440 Saran440 deleted the 13.0-fix-l10n_th_tax_report-error_install branch February 13, 2023 03:19
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.

1 participant