Skip to content

Conversation

ubernostrum
Copy link
Member

This rewrites the entire example to use the same DB names throughout,
and also is hopefully a bit more sensibly described. Additionally, the
missing import of the random module for choosing a read slave is
included in the example now.

This rewrites the entire example to use the same DB names throughout,
and also is hopefully a bit more sensibly described. Additionally, the
missing import of the random module for choosing a read slave is
included in the example now.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

return None

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this method necessary if it's always returning True?

ubernostrum added a commit that referenced this pull request Sep 8, 2012
Ticket 18657: Fix inconsistent DB names in router example.
@ubernostrum ubernostrum merged commit c26dd4f into django:master Sep 8, 2012
nanuxbe pushed a commit to nanuxbe/django that referenced this pull request Jul 2, 2016
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