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

Add helper to show CRUD history for particular object #193

Merged

Conversation

kirmalyshev
Copy link
Contributor

@kirmalyshev kirmalyshev commented Aug 3, 2021

Issue: #192
How to use:

  1. Inherit your ModelAdmin class from CRUDHistoryAdminMixin
  2. use filed crud_history_link in list_display, or in fieldsets

Example in code

class SomeModelAdmin(CRUDHistoryAdminMixin):
    list_display = (
        "id",
        "crud_history_link",
    )

@kirmalyshev kirmalyshev changed the title #192 add helper to show CRUD history for particular object Add helper to show CRUD history for particular object Aug 3, 2021
@kirmalyshev
Copy link
Contributor Author

@soynatan Did you review this PR? Maybe, you have some fix requests?

@kirmalyshev
Copy link
Contributor Author

Wanted to check -- is the PR needed? Or should I close it?

@jheld jheld self-requested a review September 4, 2022 02:59
Copy link
Collaborator

@jheld jheld left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds like the intention is to make integration with linking to crud history inside the admin more easy, so that's a positive.

@jheld jheld merged commit ca6af83 into soynatan:master Sep 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants