Skip to content
This repository has been archived by the owner on Jun 8, 2021. It is now read-only.

Add flask-migrate to enable database migration #21

Merged
merged 6 commits into from
Nov 9, 2018

Conversation

mschader
Copy link
Contributor

This will add functionability to create database migrations. The concept was taken from:
https://blog.miguelgrinberg.com/post/the-flask-mega-tutorial-part-iv-database

@fleaz
Copy link
Member

fleaz commented Oct 28, 2018

So hyped to get DB migrations. No more manual fiddeling around in the prod DB via cli :D

Will this happen automatically or do we have to write migration scripts?

@mschader
Copy link
Contributor Author

As far as i understood, these changes will be sufficient to initially crate the migrations structural folder and upgrade the database by simply using flask db <command> (e.g. flask db upgrade)

You are even able to only select single tables to migrate and stuuff like moving forward/backward in versions.

@fleaz
Copy link
Member

fleaz commented Oct 28, 2018

The failing TravisCI tests are a bug in pipenv--> #3026

@mschader mschader changed the base branch from master to development October 28, 2018 12:36
@mschader mschader changed the base branch from development to master October 28, 2018 12:36
@fleaz fleaz changed the base branch from master to development October 28, 2018 19:39
@mschader
Copy link
Contributor Author

mschader commented Nov 9, 2018

I would suggest to merge this branch already into development so we can start using it as soon as #19 is finished.

@fleaz fleaz merged commit a1cbff6 into development Nov 9, 2018
@fleaz fleaz deleted the feat/db_migration branch November 9, 2018 12:41
@fleaz
Copy link
Member

fleaz commented Nov 9, 2018

Made a squash merge because this feature branch contained strange commits because the branch was created from master instead of development.

Thanks!

fleaz added a commit that referenced this pull request Dec 10, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants