Use django 5.2 features: reverse with query/fragment, auto-imports on manage.py shell
#2428
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
niklasmohrin
approved these changes
Apr 12, 2025
Member
niklasmohrin
left a comment
niklasmohrin
left a comment
There was a problem hiding this comment.
After merging, we should also update the mention of shell_plus in the wiki
For reset_db, we could also roll our own implementation, but I don't know if we want to. On the one hand, it's really only DROP DATABASE X and CREATE DATABASE X ... (and we can remove a lot of the features of django_extensions' implementation. On the other hand, maybe we don't want to care about details with database auth and what not
richardebeling
force-pushed
the
django52
branch
from
666f0bf to
b75bae0
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
depends on #2427.
We still need django-extensions for
reset_db. Maybe we can get by without it, I think we could then drop the dependency. There is one usage inreload_testdataand one usage in the load_production_backup script in the deployment repo.