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

Upgrade to work with Django 1.5 and tastypie 0.9.12 #35

Merged
merged 5 commits into from May 1, 2013

Conversation

Projects
None yet
3 participants
@palewire
Contributor

palewire commented Mar 23, 2013

This requires

  • Upgraded setup.py to pull the latest tastypie
  • Removing an boundaryservice admin.py registration of tastypie model that tastypie now registers on its own
  • A further hack of tastyhacks.py's get_resource_uri method
@palewire

This comment has been minimized.

Show comment
Hide comment
@palewire

palewire Mar 23, 2013

Contributor

Aimed at tickets #32 and #33

Contributor

palewire commented Mar 23, 2013

Aimed at tickets #32 and #33

A refinement of the #32 and #33 patch that avoid the pitfalls that ca…
…used #38. This is all caused by the self.get_resource_uri(None) called on line 1262 of the resources.py file in tastypie
@palewire

This comment has been minimized.

Show comment
Hide comment
@palewire

palewire Apr 17, 2013

Contributor

This should be a little sturdier now.

Contributor

palewire commented Apr 17, 2013

This should be a little sturdier now.

ryanmark added a commit that referenced this pull request May 1, 2013

Merge pull request #35 from datadesk/upgrade_tastypie
Upgrade to work with Django 1.5 and tastypie 0.9.12

@ryanmark ryanmark merged commit 1db5034 into newsapps:master May 1, 2013

@eyeseast

This comment has been minimized.

Show comment
Hide comment
@eyeseast

eyeseast Nov 1, 2013

This is breaking with the latest stable (pypi) version of tastypie. It raises AlreadyRegistered: The model ApiKey is already registered. Happens regardless of ordering in INSTALLED_APPS, the second one just triggers the error.

eyeseast commented on 5908126 Nov 1, 2013

This is breaking with the latest stable (pypi) version of tastypie. It raises AlreadyRegistered: The model ApiKey is already registered. Happens regardless of ordering in INSTALLED_APPS, the second one just triggers the error.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment