-
Notifications
You must be signed in to change notification settings - Fork 104
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
Update for django-reversion 2.04 #73
Conversation
mshannon1123
commented
Jun 28, 2016
- Sorry for the large PR, but included is some pep8 cleanup and optimizations
- Unit Tests pass locally
- Testing ok with django-reversion 2.04 and Oracle as DB
Thanks for your contribution! Please change .travis.yml, so that the tests runs also against django-reversion v2 ;) |
@@ -91,9 +84,9 @@ def __eq__(self, other): | |||
return False | |||
|
|||
# see - https://hynek.me/articles/hasattr/ | |||
internal_type = getattr(self.field,'get_internal_type',None) | |||
if internal_type is None or internal_type() == "ForeignKey": # FIXME! |
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.
Why remove comments?
19ee6cc
to
c427a18
Compare
@jedie I pushed up and corrected a cross-compatibility error. Let me know if you see anything else. |
|
||
if missing_objects: | ||
result.append("missing m2m objects: %s" % repr(missing_objects)) | ||
result.append("missing m2m objects: %s" % repr(missing_objects)) |
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.
Seems like a intention error?!?
Just a ping to remind if you have a moment to review this change. |
This will be great. |
- DJANGO='1.9' REVERSION='1.10' EXTRA='diff-match-patch' | ||
- DJANGO='1.9' REVERSION='2.0.4' EXTRA='diff-match-patch' |
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 think "2.0.4" is too limited. (Current django-reversion version is 2.0.6)
Why not use "2.0" ?
Think it makes sense to support only django-reversion >= 2.x |
Please take a look at #76 ;) |
v0.7.0 released on PyPi and will support django-reversion v2.0.x |
Looked great, thanks for the work! |