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

Closed

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
Copy link

stale bot commented May 22, 2024

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed within 3 days if no further activity occurs, but it only takes a comment to keep a contribution alive :) Also, even if it is closed, you can always reopen the PR when you're ready. Thank you for contributing.

@stale stale bot added the inactive label May 22, 2024
@casesolved-co-uk
Copy link
Contributor Author

@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

No

This PR is available here: https://github.com/CaseSolvedUK/frappe

@casesolved-co-uk casesolved-co-uk deleted the doc_get_filters branch May 22, 2024 14:54
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 inactive
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug in core document.get
2 participants