-
-
Notifications
You must be signed in to change notification settings - Fork 347
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
Reset django migrations, document upgrade #4145
Conversation
@kgodey You might want to read through the "upgrade" docs portion of this PR. |
If you have a Docker compose installation (including one from the guided script), run the command below: | ||
|
||
``` | ||
docker compose -f /etc/mathesar/docker-compose.yml up --pull always -d |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mathemancer We would be moving to a different repository in Docker i.e. mathesar/mathesar
instead of mathesar/mathesar-prod
, so this wouldn't work.
We would have to ask users to modify their docker-compose file or get the latest one from our repo.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch, thanks
Co-authored-by: Anish Umale <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good overall, I've made minor suggestions that needs your attention @mathemancer.
docs/docs/releases/0.2.0.md
Outdated
python manage.py compilemessages | ||
``` | ||
|
||
1. Update Mathesar functions on the database: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't install/update any functions using install.py anymore, I am skeptical whether we even need this step.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're right. Removed!
Okay, I'm going to merge this in interest of time. @kgodey We can add more docs updates to the release branch after it's cut. |
This PR:
Technical details
The documentation walks the user through knocking out their old
mathesar_django
database, recreating it, and then proceeding with the upgrade.Checklist
Update index.md
).develop
branch of the repositoryvisible errors.
Developer Certificate of Origin
Developer Certificate of Origin