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 ignoring the get_queryset() of ModelAdmin and get_base_queryset()… #10455

Closed

Conversation

engAmirEng
Copy link

Fixes wagtail-nest/wagtail-modeladmin#21

none of the drived ViewClassed of InstanceSpecificView respected the get_queryset methods before

  • Do the tests still pass?[^1]
  • Does the code comply with the style guide?
  • For Python changes: Have you added tests to cover the new/fixed behaviour?

since InstanceSpecificView drives from WMABaseView I handled the issue as it is handled in WMABaseView

@squash-labs
Copy link

squash-labs bot commented May 19, 2023

Manage this branch in Squash

Test this branch here: https://engamirengbug-fixmodeladmin-qu-v5slo.squash.io

@laymonage
Copy link
Member

Hello, I'm closing since we no longer accepting PRs for new ModelAdmin features in Wagtail as per wagtail/rfcs#85. If you'd like, you could create a new PR to the wagtail-modeladmin repository.

And for this PR in particular, per wagtail-nest/wagtail-modeladmin#21 (comment) this is an enhancement rather than a bug fix, so ideally this should be accompanied with documentation.

Thank you!

@laymonage laymonage closed this Jul 26, 2023
@lb- lb- removed their request for review January 20, 2024 12:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Overriding modeladmin's get_queryset not respected by Edit-, Inspect & DeleteView
3 participants