search

OCA / l10n-thailand

Thai localization
GNU Affero General Public License v3.0
50 stars 67 forks source link

[15.0][FIX] l10n_th_account_tax_report: query mistake #412

Closed Saran440 closed 11 months ago

Saran440 commented 12 months ago

Fix bug query account tax report when it has a report late.

This PR is fixed query following:

Example Case: We have tax invoice 7 document

ID  Tax Invoice Late    Report Date
1   09/11/23    0   09/11/23
2   11/10/23    1   30/11/23
3   30/11/23    0   30/11/23
4   30/10/23    0   30/10/23
5   03/09/23    2   30/11/23
6   14/04/22    4   14/08/22
7   16/08/23    2   31/10/23

Filter thai tax report 01/11/23 - 30/11/23 it should be result ID 1, 2, 3, 5

Filter thai tax report 01/11/23 - 15/11/23 it should be result ID 1, 2, 5

Filter thai tax report 30/10/23 - 01/11/23 it should be result ID 2, 4, 5, 7

OCA-git-bot commented 12 months ago

Hi @kittiu, some modules you are maintaining are being modified, check this out!

kittiu commented 11 months ago

/ocabot mrege patch

OCA-git-bot commented 11 months ago

Hi @kittiu. Your command failed:

Invalid command: mrege.

Ocabot commands

More information

OCA-git-bot commented 11 months ago

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

Saran440 commented 11 months ago

/ocabot merge patch

OCA-git-bot commented 11 months ago

What a great day to merge this nice PR. Let's do it! Prepared branch 15.0-ocabot-merge-pr-412-by-Saran440-bump-patch, awaiting test results.

OCA-git-bot commented 11 months ago

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