-
-
Notifications
You must be signed in to change notification settings - Fork 151
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
Feature/history #349
Feature/history #349
Conversation
@@ -2,13 +2,107 @@ | |||
""" Boilerplate for now, will serve a purpose soon! """ | |||
from __future__ import division, absolute_import, unicode_literals | |||
|
|||
from django.conf import settings | |||
from django.contrib.contenttypes.models import ContentType | |||
from django.contrib.admin.util import quote |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I hate to be a jerk, but this single line of code means I can't accept this pull request.
Please read https://django-admin2.readthedocs.org/en/latest/design.html, especially the constraints.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
God, you jerk 👯
We'll port that function over soon.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you mean jerk or krej? Krej, as you know, is the name of the sauce that Pac Rim Kaiju put on their giant hamburgers.
👊
Feature/history - Merging anyway. I'll pull out the auth link tomorrow.
Model History
LogEntry
model from the original Django admin.My Recent Actions
module to the home page.