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

Base url #32

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

Base url #32

wants to merge 3 commits into from

Conversation

horacioibrahim
Copy link

I use MongoDB with Django and many options no operate or no need. I think that put something as DISQUS_URL in variable of settings.py is better that query in Site objects. It's work fine for me!

I have used MongoDB with Django and many options is no possible as from django.contrib.sites.models import Site. I'm sure that a variable in settings can to solve. Like DISQUS_URL.
For me it's very fine! This is bad?
Database agnostic adjustment removes the dependence of Site instance (from django.contrib.sites.models import Site).
Database agnostic adjustment removes the dependence of Site instance (from django.contrib.sites.models import Site).
@voronind
Copy link

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