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

Include Phinx migrations from extensions #323

Merged
merged 2 commits into from
Jun 27, 2017
Merged

Include Phinx migrations from extensions #323

merged 2 commits into from
Jun 27, 2017

Conversation

stevenrombauts
Copy link
Member

Resolve #318

@stevenrombauts stevenrombauts added this to the 2.0.0-beta milestone Jun 20, 2017
@stevenrombauts stevenrombauts self-assigned this Jun 20, 2017
@stevenrombauts stevenrombauts mentioned this pull request Jun 20, 2017
Closed
@johanjanssens
Copy link
Member

@allanpilarca Can you test this please for instructions see: #318 (comment)

johanjanssens
johanjanssens approved these changes Jun 26, 2017
View reviewed changes
Copy link
Member

@johanjanssens johanjanssens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@stevenrombauts Code is looking good. Asked @allanpilarca to review.

allanpilarca
allanpilarca approved these changes Jun 27, 2017
View reviewed changes
Copy link
Contributor

@allanpilarca allanpilarca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested. Working as expected. :)

@johanjanssens johanjanssens merged commit 1c7643f into master Jun 27, 2017
@johanjanssens johanjanssens deleted the feature/318-phinx branch June 27, 2017 05:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@johanjanssens johanjanssens johanjanssens approved these changes

@allanpilarca allanpilarca allanpilarca approved these changes

Assignees

@stevenrombauts stevenrombauts

Labels
None yet
Projects
None yet
Milestone
2.0.0
Development

Successfully merging this pull request may close these issues.
3 participants
@stevenrombauts @johanjanssens @allanpilarca