Skip to content
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

Database migrations #79

Open
edsu opened this issue May 9, 2020 · 1 comment
Open

Database migrations #79

edsu opened this issue May 9, 2020 · 1 comment

Comments

@edsu
Copy link
Member

edsu commented May 9, 2020

The next version of diffengine will require some database modifications for existing installs. peewee supports migrations. I think we have an example of one in init.py but maybe we should pull these out into a separate module?

I can test the migrations on a v0.2.7 database that I have.

@edsu edsu changed the title Confirm migrations Confirm migrations work May 9, 2020
@edsu edsu changed the title Confirm migrations work Database migrations May 9, 2020
@nahuelhds
Copy link
Contributor

nahuelhds commented May 26, 2020

I was going to create an issue because I couldn't make to work the tweeting when using PostgreSql and it was because I had an older version of the Diff model schema and I needed to add the emailed column that was created at #77.

I lost a lof of debugging hours with this one :(

I say we create a /migrations/{version}/{engine}.sql file where the migration are written, so the user know that need to execute that one on the database he's using.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants