History view pagination

[jurrian]

Description

This will add pagination and filtering to the simple history admin view. This is needed because it will allow more performance views on instances with many historical records. Currently it is very likely to timeout depending on the settings.

It uses the same pagination as the default in Django.

Related Issue

This PR replaces this stale PR: #1220
Most of the text of this PR is copied from the stale one.

Closes #1117 and closes #1219.

Motivation and Context

This is needed because it will allow more performance views on instances with many historical records. Currently it is very likely to timeout depending on the settings .

How Has This Been Tested?

Added a few extra tests on top of what already existed in the stale PR.

Screenshots (if appropriate):

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• I have run the pre-commit run command to format and lint.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• I have added my name and/or github handle to AUTHORS.rst
• I have added my change to CHANGES.rst
• All new and existing tests passed.

[jurrian]

@ddabble @RumitAP I've created a new PR based on #1220 with some additional changes.

[jurrian]

@ddabble could you have a look at this? It's basically the same as #1220 that you reviewed, with a small addition to make it work.

[diogosilva30]

Any update on this? Do you need any help?

[jurrian]

@ddabble ?

[ddabble]

@jurrian Sorry for the (very) late reply; I'm busy with some of the other PRs, currently, but this is definitely on my TODO list 🙂

[ferran-gutierrez]

Hi guys!

How is going this feature? I'm super engaged with that, for us would be an amazing deliveryHi guys!

How is going this feature? I'm super engaged with that for us will be an amazing delivery

[jurrian]

Only needs review at the moment. Code should be complete, but needs to be tested and merged.

[tim-schilling]

This looks good to me. It seems like it's a bit old now and needs to be rebased on main. Is that something you may have time for yet?

[jurrian]

@tim-schilling I have done the rebase and did some minor tweaks. Most notably historical_records is now page_obj to avoid confusion about the object. I tried to keep it in line with the Django docs and django/contrib/admin/templates/admin/object_history.html pagination html.

[tim-schilling]

This looks good to me! Thank you @jurrian!