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

Document TEST_MIRROR caveat #14811

Closed
wants to merge 1 commit into from
Closed

Conversation

christianbundy
Copy link
Contributor

Problem: This setting only works in transactions but that isn't documented.

Solution: Add caveat to documentation.

Link: https://code.djangoproject.com/ticket/23718

Problem: This setting only works in transactions but that isn't documented.

Solution: Add caveat to documentation.

Link: https://code.djangoproject.com/ticket/23718
@felixxm
Copy link
Member

felixxm commented Nov 19, 2021

@christianbundy Thanks for this patch 👍 IMO we should also add a note to the "Testing primary/replica configurations" section.

@felixxm
Copy link
Member

felixxm commented Jul 1, 2022

Closing due to inactivity.

@felixxm felixxm closed this Jul 1, 2022
@christianbundy
Copy link
Contributor Author

I think that this is a net positive, can we merge as-is or is it incomplete without adding a note somewhere else?

@stickperson
Copy link

Any reason this has not been merged? I spent a non-trivial amount of time figuring this out today.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants