-
Notifications
You must be signed in to change notification settings - Fork 897
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
Initial implementation of a breakpoint for migration rollback. #497
Conversation
51904d8
to
8b5dd71
Compare
6907de0
to
137a177
Compare
137a177
to
cbbe386
Compare
eb52607
to
5f1a6a7
Compare
c672583
to
16f536b
Compare
16f536b
to
5374f27
Compare
5374f27
to
6c54782
Compare
6c54782
to
cca7a70
Compare
b8badb9
to
4b5fdee
Compare
4b5fdee
to
983a43c
Compare
waiting to see if theres further support from the community |
41e74fa
to
3d8af56
Compare
3d8af56
to
a896c73
Compare
a896c73
to
d544759
Compare
d544759
to
be8090b
Compare
be8090b
to
2340a44
Compare
This is a really good feature and I would love to see if incorporated into phinx! It means you can have a "safety net" so you don't accidentally roll back all your database changes. It also means that when you start a new task, you can put a breakpoint in on the "current" change, and it's really easy to roll back to that point while you're working on that task. |
b5474d5
to
4986783
Compare
349332d
to
2966e04
Compare
Old domain of box project is squatted!
Add support for type YEAR
Change box project lint to actual github page.
MySQL adapter add support for type YEAR - Issues cakephp#551
… a better placement.
Updated tests to support statuses requirement of needing the full row to display the breakpoint.
…list of version numbers, the other returns the full row (along with the breakpoint flag). Updated the testRollbacksByDate test to mock the getFullVersions() call appropriately. Updated the testMigrationsByDate test to verify that getFullVersions() is not called as the full record is not needed as part of the migration run.
2966e04
to
7c08008
Compare
Thank you for contributing! The branch you made this PR against is no longer accepting patches. This probably happened because we didn't merge your changes fast enough. 😭 Please create a new PR against the current default branch if you want to see these changes merged. |
See #494 for the issue.