Fixed #30601 -- Added warning in docs #16848
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
Hello! Thank you for your contribution 💪
As it's your first contribution be sure to check out the patch review checklist.
If you're fixing a ticket from Trac make sure to set the "Has patch" flag and include a link to this PR in the ticket!
If you have any design or process questions then you can ask in the Django forum.
Welcome aboard ⛵️!
| If your code uses caching or proactively updates data in the cache, consider | ||
| disabling ATOMIC_REQUESTS to avoid unexpected behavior. Similarly, if your | ||
| code assumes that database changes are visible outside of the transaction, | ||
| you may need to update your code to take transactions into account. |
There was a problem hiding this comment.
Have you seen Simon's suggestion?
There was a problem hiding this comment.
My first reaction is that this warning says the same thing multiple times without providing any feasible solution 🤔 e.g. it doesn't mention transaction.on_commit.
|
@angelicaferlin Do you have time to keep working on this? |
Sorry, I don´t have time and have deassigned myself from the ticket. Anyone interested can pick up from here! |
|
Closing following the message from the author, and resetting flags in the ticket so hopefully someone else picks this up. |
3 participants
Proposed solution for https://code.djangoproject.com/ticket/30601