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

perf: Optimize get_attribute_filters (#26729) #27931

Conversation

marination
Copy link
Collaborator

@marination marination commented Oct 12, 2021

Backport of #26729

  • Manual Testing

* perf: Optimize get_attribute_filters

* fix: handle when filter attributes are undefined

* chore: unused imports

Co-authored-by: Ankush Menat <ankush@iwebnotes.com>
@marination marination added the CI-failing Unit tests or patch tests are failing. label Oct 13, 2021
The function returns strings directly instead of objects now.
@ankush ankush removed the CI-failing Unit tests or patch tests are failing. label Oct 21, 2021
@marination marination merged commit 67e5e95 into frappe:version-13-hotfix Oct 27, 2021
asoral pushed a commit to asoral/erpnext that referenced this pull request Nov 12, 2021
…e-filters-hotfix

perf: Optimize get_attribute_filters (frappe#26729)
marination added a commit to marination/erpnext that referenced this pull request Feb 1, 2022
mergify bot pushed a commit that referenced this pull request Feb 3, 2022
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 10, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants