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

Use Django Database routing #32

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

GeorgeLubaretsi
Copy link

In our project we use multiple databases. They are chosen based on Model's app name.

Here's the documentation:
https://docs.djangoproject.com/en/1.11/topics/db/multi-db/#using-routers

This change makes sure routers are taken into account when creating a connection

@scott-w
Copy link
Contributor

scott-w commented Nov 16, 2017

Hi @GeorgeLubaretsi, thanks for this!

Are you able to pull together a test, or modify the tests, to demonstrate this working?

@scott-w
Copy link
Contributor

scott-w commented Dec 22, 2017

Hi @GeorgeLubaretsi, I'd still like to merge this in. If you'd like some help putting a test case together, I'm more than happy to work through it with you =)

@GeorgeLubaretsi
Copy link
Author

Hey @scott-w I really don't have much time at the moment :( I'll try to come up with something over the next weekend

@scott-w
Copy link
Contributor

scott-w commented Jan 8, 2018

Hi @GeorgeLubaretsi, no worries. Just keep me updated and I'll be happy to put some time aside to work with you.

@mikicz
Copy link

mikicz commented Nov 22, 2021

My fork now this supports this as of version 0.8.0. See https://github.com/mikicz/django-pgviews and https://pypi.org/project/django-pgviews-redux/.

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.

3 participants