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: refactor document.get definition (#26280) #26281

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

casesolved-co-uk
Copy link
Contributor

@casesolved-co-uk casesolved-co-uk commented May 1, 2024

fix: change document.get definition to match dict.get and add positional deprecation warning to the filters keyword

fixes: #26280

Have attempted to fix all core files using the new filters= arg

See also #26279

version-14-hotfix
version-15-hotfix

@casesolved-co-uk casesolved-co-uk requested a review from a team as a code owner May 1, 2024 18:08
@casesolved-co-uk casesolved-co-uk requested review from akhilnarang and removed request for a team May 1, 2024 18:08
@github-actions github-actions bot added the add-test-cases Add test case to validate fix or enhancement label May 1, 2024
@casesolved-co-uk casesolved-co-uk force-pushed the doc_get_filters branch 5 times, most recently from 1be3e62 to 61feaff Compare May 1, 2024 19:39
@ankush
Copy link
Member

ankush commented May 3, 2024

@casesolved-co-uk can you please raise PRs from personal account instead of org forks

https://github.com/frappe/erpnext/wiki/Pull-Request-Checklist#maintainer-edits

…nal deprecation warning to the filters keyword

fix: doc.get filters arg usage
fix: linting
fix: customize_form
fix: desk and doctype
fix: init, doctype, customize, todo, form
fix: website
fix: change inspect for deprecation_warning
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
add-test-cases Add test case to validate fix or enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug in core document.get
2 participants