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

fix: filter by cost center in trial balance #36324

Merged

Conversation

GursheenK
Copy link
Member

Problem
Filtering the Trial Balance Report resulted in the following error -

Screenshot 2023-07-26 at 11 08 48 AM

Solution
Changed the where clause for cost centers in the Trial Balance query to filter the list of cost centers correctly.

@github-actions github-actions bot added the needs-tests This PR needs automated unit-tests. label Jul 26, 2023
@deepeshgarg007
Copy link
Member

@Mergifyio backport version-14

@mergify
Copy link
Contributor

mergify bot commented Jul 26, 2023

backport version-14

✅ Backports have been created

@deepeshgarg007 deepeshgarg007 merged commit 4b49864 into frappe:version-14-hotfix Jul 26, 2023
deepeshgarg007 added a commit that referenced this pull request Jul 26, 2023
fix: filter by cost center in trial balance (#36324)
@frappe-pr-bot
Copy link
Collaborator

🎉 This PR is included in version 14.33.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
needs-tests This PR needs automated unit-tests. released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants