Skip to content
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

- Updated for Django 1.6+ (changed ADMIN_MEDIA_PREFIX to STATIC_URL) #3

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

adamestein
Copy link

  • Updated the PopUpBaseWidget to supply the anchor link if no template name is given, removing the need for an external file (also simplified expressing the JavaScript file needed for the media)
  • Made the 'popup.html' name an argument in add_new_model() in case somebody wants to override the name
  • Updated version number
  • Cleaned up PEP8 warnings / slight refactoring

adamestein added 4 commits May 7, 2016 22:46
- Updated the PopUpBaseWidget to supply the anchor link if no template name is
  given, removing the need for an external file (also simplified expressing
  the JavaScript file needed)
- Made the 'popup.html' name an argument in add_new_model() in case somebody
  wants to override the name
- Updated version number
- Cleaned up PEP8 warnings / slight refactoring
…ng 'fields' or 'exclude' in Meta

and using render_to_response() instead of the new render())
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.

1 participant