Skip to content

Conversation

@javiercf
Copy link

@javiercf javiercf commented Nov 5, 2014

removed defaults parameter in get_urls method that crashes the SingletonModelAdmin Class

removed defaults parameter in get_urls method that crashes the SingletonModelAdmin Class
@defbyte
Copy link
Owner

defbyte commented Nov 5, 2014

Hi @javiercf , thanks for submitting the pull request!

We need to make sure we maintain backwards compatibility, as there are a few legacy projects that use this package which are stuck on older version of Django (one of which is on Django 1.3.x).

Perhaps with a simple ImportError check?

added an ImportError check to maintain backwards compatibility
@javiercf
Copy link
Author

javiercf commented Nov 5, 2014

hey @defbyte you're totally right, I just added the ImportError check, hope it helps!

defbyte added a commit that referenced this pull request Nov 5, 2014
Update admin.py compatibility with Django 1.6, maintaining backwards compatibility
@defbyte defbyte merged commit f26e017 into defbyte:master Nov 5, 2014
@defbyte
Copy link
Owner

defbyte commented Nov 5, 2014

@javiercf I'll need to update the version number, repackage, and upload to PyPi

@defbyte
Copy link
Owner

defbyte commented Nov 5, 2014

Release created and PyPi updated: https://pypi.python.org/pypi/django-singleton/0.1.9

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