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][MIG] l10n_th_withholding_tax_report #81

Merged

Conversation

kongrattapong
Copy link
Contributor

@kongrattapong kongrattapong commented Jan 20, 2020

Work in progress

========================

  • PDF report
  • HTML report
  • XLSX report - waiting module report_xlsx_helper migration to V13
  • TXT report - waiting module report_csv migration to V13

kongrattapong and others added 8 commits January 20, 2020 17:25
report_backend widget.

These missing two function in the widget cause two bugs in the Odoo
Client. (tested on Odoo Enterprise Version).

1. The bug occor when we are on Withholding Tax Report view page, then "click" on "Home
menu" (located on the top left of the screen). This bug is caused by
missing of on_detach_callback.

2. The bug occor when we are on Withholding Tax Report view page and then we
"refresh" the page (pressing F5 or CTRL+R). This bug is caused by
missing of on_attach_callback.
@kittiu
Copy link
Sponsor Member

kittiu commented Jan 21, 2020

@kongrattapong before I move on with the review. I think, to generate TXT file, we should use report_csv instead of xlsx_import_export. This is to make things more inline (qweb, xlsx, txt to use same technique).

Can you,

Thanks!

@kongrattapong kongrattapong force-pushed the 13.0-mig-l10n_th_withholding_tax_report branch 3 times, most recently from 5539cc7 to d4fe32b Compare January 31, 2020 07:23
@kongrattapong kongrattapong force-pushed the 13.0-mig-l10n_th_withholding_tax_report branch from d4fe32b to 864bc93 Compare January 31, 2020 07:28
@kongrattapong
Copy link
Contributor Author

kongrattapong commented Jan 31, 2020

@kittiu Sorry for late reply.
Migrate report_csv : OCA/reporting-engine#367
and I adjust code to use that module already.

thank for your review.

@kittiu
Copy link
Sponsor Member

kittiu commented Jan 31, 2020

Please rebase

@kongrattapong
Copy link
Contributor Author

rebase already.

@kittiu
Copy link
Sponsor Member

kittiu commented Feb 21, 2020

@kongrattapong, we can continue this case, only report_csv left, we will try to push it up.

@kittiu
Copy link
Sponsor Member

kittiu commented Mar 20, 2020

@kongrattapong all modules needed are merged. Can you rebase please?
I can do more review and move this forward.

oca-travis and others added 2 commits March 24, 2020 06:04
Tax report to be more correct, when working with account_document_reversal
kittiu and others added 22 commits April 24, 2020 19:04
Signed-off-by kittiu
Signed-off-by kittiu
File "/home/odoo/src/user/l10n_th_tax_invoice/models/account_move.py", line 98, in create
vals["payment_id"] = self._context["payment_id"]
TypeError: list indices must be integers or slices, not str
Signed-off-by kittiu
report_backend widget.

These missing two function in the widget cause two bugs in the Odoo
Client. (tested on Odoo Enterprise Version).

1. The bug occor when we are on Withholding Tax Report view page, then "click" on "Home
menu" (located on the top left of the screen). This bug is caused by
missing of on_detach_callback.

2. The bug occor when we are on Withholding Tax Report view page and then we
"refresh" the page (pressing F5 or CTRL+R). This bug is caused by
missing of on_attach_callback.
…ecosoft-odoo/l10n-thailand into 13.0-mig-l10n_th_withholding_tax_report

ู
@kittiu
Copy link
Sponsor Member

kittiu commented May 12, 2020

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 13.0-ocabot-merge-pr-81-by-kittiu-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 244bd17 into OCA:13.0 May 12, 2020
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at b44c943. Thanks a lot for contributing to OCA. ❤️

@Saran440 Saran440 deleted the 13.0-mig-l10n_th_withholding_tax_report branch February 20, 2023 09:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants