fix: bypass error checking for initial migration process#2092
Closed
jmcruz14 wants to merge 1 commit into
Closed
Conversation
* This change resolves a bug observed when attempting to run the dev server of the repository and an OperationalError with the message no such table: auth_permission is returned
Contributor
Author
|
Closing PR since #2072 resolves this. Thanks! |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
When attempting to test the available dev-server within the repo via
uv run -- python manage.py runserver, Django returns the following error message:django.db.utils.OperationalError: no such table: auth_permission. Attempting to runpython manage.py migratebeforerunserverleads to the same issue.Currently, the only way to overcome this is to run the
migratecommand with the--skip-checksflag included.To ensure consistency with the listed documentation, the fix implements a
try/exceptbypass withinsrc/unfold/checks.pyto ensure developers can have an easy installation process for the repo on their device.Test plan
OS: Sequoia 15.7.1
Python Version: 3.12.2
Django Version: 5.2.14
db.sqlite3withintests/serveris deleted to ensure a fresh installation.uv run -- python manage.py migrateto check if the code change is met.uv run -- pytest .was run to ensure the changes do not break anything else, with the tests executed from my end having shown417 passed in 19.74swith a total coverage of95.75%.Use of AI
docs/development/index.mdcould not be followed through properly.