search

darklow / django-suit

Modern theme for Django admin interface
http://djangosuit.com/
Other
2.33k stars 705 forks source link

history links broken? #502

Open ed4g opened 8 years ago

ed4g commented 8 years ago

I seem to be getting the same sort of bug that was reported in https://github.com/darklow/django-suit/issues/457, but I'm in django 1.9.2 and the current version of suit (2.18).

I am using simple_history for history tracking (for the time being) because it captures changes no matter where they occur (though this is imperfect still and I may change it.)

django-simple-history>=1.8.1 django-suit>=0.2.18

Reverse for 'prancer_meta_historicalstudy_add' with arguments '()' and keyword arguments '{}' not found. 0 pattern(s) tried: [] Request Method: GET Request URL: http://localhost:8001/admin/prancer_meta/study/2/history/2/ Django Version: 1.9.2 Exception Type: NoReverseMatch Exception Value: Reverse for 'prancer_meta_historicalstudy_add' with arguments '()' and keyword arguments '{}' not found. 0 pattern(s) tried: []

When I comment suit out of INSTALLED_APPS and go back to vanilla django admin, and do nothing else, I hit the same link and get the history page without issue, at the following URL:

http://localhost:8001/admin/prancer_meta/study/2/history/2/

Oddly enough, when I go back to suit and paste that parameter into the browser, I still get the error page.

Is this an old bug resurfaced, or perhaps an issue that I've created somehow?

ed4g commented 8 years ago

Also - a question - in the demo history you are able to show what field was changed, not just "changed" as simple_history does - how are you doing that? Would it be possible to show something like:

color was changed from black to blue

That would be hugely valuable IMO