-
Notifications
You must be signed in to change notification settings - Fork 30
INTPYTHON-759 Deprecate parse_uri() #404
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
Conversation
dddaf26 to
f8c9528
Compare
| DATABASES = { | ||
| 'default': { | ||
| 'ENGINE': 'django_mongodb_backend', | ||
| 'HOST': 'mongodb://localhost:27017/', |
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.
Did I notice correctly recently that HOST can be omitted and localhost is chosen by default? If so, probably OK? If not, never mind.
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.
Correct. I raised that at: mongodb-labs/django-mongodb-project#15 (comment). This example is the old template and the new one. We could replace "localhost:27017" with "..." if that seems better.
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.
Got it, cool. I'd say let's leave it for now since we're trying to emphasize HOST=URI. Maybe we'll update the project template later to only include the ENGINE and document the other options.
No description provided.