Closed sunnysideup closed 1 month ago
You seem to have removed the pull request template from the description when you created this. I've added it back. Please fill in the pull request template in full, including linking to an issue and creating a new issue if one doesn't exist yet.
This is all the work I can do on this ticket. Please use the code if you like or, if not, someone else can pick up the ticket from here and make it their commit.
If by this you mean you aren't able to commit to making changes when they're requested, please just close this pull request.
Description
If the total list has less than the items per page you need to make sure that you set the items per page to be the total number of items otherwise, you still get for example 20 items in the gridfield, but the paginator info will say 1 of 1. Basically, the paginator overrides the limit / total amount of items, but instead the paginator limit should be equal to the total number available in the full list (in this case).
This is all the work I can do on this ticket. Please use the code if you like or, if not, someone else can pick up the ticket from here and make it their commit.
I understand that perhaps you should not be allowed to limit a list for ModelAdmin, but I feel there is value in doing so. I could be wrong. Maybe we could start with some feedback on this first.
Manual testing steps
Issues
See above
Pull request checklist
I don't have time to double-check this. This is a best effort approach. Lets first consider if this is worthwhile at all.
Also, as I have said before, if you find that I create more noise than help then just auto-delete any issues / commits I post. I am very happy for you to do so as it would save me the effort of posting it at all. However, I do not have enough time to follow through on all of this (as no one pays me for this time and I still have to provide for my family) so it has to be a halfway house.