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.

Sign up for GitHub

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

Jump to bottom

[5209][ADD] date_range, account_financial_report #36

Open
wants to merge 4 commits into
base: 16.0
Choose a base branch
from
Open

[5209][ADD] date_range, account_financial_report #36

wants to merge 4 commits into from

Conversation

AungKoKoLin1997
Copy link
Contributor

@yostashiro yostashiro changed the title [5209][ADD] account_financial_report [5209][ADD] date_range, account_financial_report Mar 5, 2025
@yostashiro
Copy link
Member

@AungKoKoLin1997 Can you please look into the CI error?

@AungKoKoLin1997
Copy link
Contributor Author

AungKoKoLin1997 commented Mar 6, 2025 •
edited
Loading

Can you please look into the CI error?

I added test-requirements.txt to add odoo_test_helper.
But there is another issue. That is auditlog causes failing the test cases of account_financial_report.
I don't have quick solution and will take time if I find out.

@yostashiro
Copy link
Member

@AungKoKoLin1997 Let me check if we could take auditlog off this repo.

@yostashiro
Copy link
Member

@AungKoKoLin1997 Let me check if we could take auditlog off this repo.

Looks like we can't. Can you please sort out the CI issue with high priority, as it's blocking the project proceedings?

@AungKoKoLin1997 AungKoKoLin1997 force-pushed the 5209-add-account_financial_report branch from 7d5e208 to 2c5d425 Compare March 12, 2025 03:36
@AungKoKoLin1997
Copy link
Contributor Author

Added new commit to fix the CI as a temporary fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yostashiro yostashiro Awaiting requested review from yostashiro

@kanda999 kanda999 Awaiting requested review from kanda999

@nobuQuartile nobuQuartile Awaiting requested review from nobuQuartile

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@AungKoKoLin1997 @yostashiro