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

modified migration files #312

Closed
wants to merge 2 commits into from

Conversation

omricohen
Copy link

I removed all the migration updates and moved them into 4 their respective create migrations. This will greatly simplify new installs.

@msevestre
Copy link
Collaborator

I fixed the build. Should be back in businesss soon

@msevestre
Copy link
Collaborator

@omricohen could you somehow update that PR to trigger the build again?Thanks

@omricohen
Copy link
Author

@msevestre I did -- still failed

@msevestre
Copy link
Collaborator

msevestre commented Mar 7, 2017 via email

@jipiboily
Copy link
Owner

yeah, I want to make the build pass again shortly. :)

@jipiboily
Copy link
Owner

@omricohen while working on making the build pass (#301) I had to resetup Monologue locally, and the migrations worked perfectly fine for me.

What is this PR trying to fix/change exactly? Not 100% clear to me.

Thanks!

@jipiboily
Copy link
Owner

That's going to be more complicated than I thought. See #301 please. Happy to pay $100 if someone wants to get Monologue back on track.

@jipiboily
Copy link
Owner

Actually, see #314 instead.

@omricohen
Copy link
Author

@jipiboily This PR makes it so that you can rollback a monologue install. The current migrations do not allow for a rollback. Plus, not point in keeping so many migration 'change' files, makes it harder to maintain and understand what is going on.

@msevestre
Copy link
Collaborator

I am going to close this PR as it is polluting my Review list and is clearly outdated :)

@msevestre msevestre closed this Jun 28, 2023
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

Successfully merging this pull request may close these issues.

3 participants