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

Add a migration script for upgrading to Stream from 1.4.9 #749

Merged
merged 2 commits into from Sep 2, 2015

Conversation

Projects
None yet
3 participants
@lukecarbis
Copy link
Contributor

commented Sep 1, 2015

@MarioKnight Can you please test this branch out on one of your version 1.4.9 installs? Please take a backup first.

Resolves #733.

@MarioKnight

This comment has been minimized.

Copy link

commented Sep 1, 2015

I have performed the following steps on a 1.4.9 install:

  • Downloaded ZIP of this pull request
  • De-activated 1.4.9, installed ZIP of this pull request and activated
  • It seemed to take a bit longer for the plugin activate page to load, however no errors were produced when it did
  • Going to the Stream page does not prompt for an update

On the Stream page itself, all of the data look as it should except for the date field. Every entry contains the following in the date:

14 years ago
-0001/11/29
07:00:00 PM

I've gone ahead and reverted back to 1.4.9 and restored the database from the backup.

Thank you for your continued work on this. Please let me know if you'd like for me to do anything else on my end.

@Japh

This comment has been minimized.

Copy link

commented Sep 1, 2015

fwiw, I had problems migrating (from v2 to v3) when I was deactivating v2 and activating v3. But when I just updated the plugin, it worked flawlessly.

Not sure if that's what's going on here, but I thought it might be worth mentioning.

@lukecarbis

This comment has been minimized.

Copy link
Contributor Author

commented Sep 1, 2015

@MarioKnight Great! That's what is supposed to happen (except the date thing). I'll try to fix that part today.

@lukecarbis

This comment has been minimized.

Copy link
Contributor Author

commented Sep 1, 2015

screen shot 2015-09-02 at 09 44 07

Just for lolz. I feel like an archaeologist.

@lukecarbis

This comment has been minimized.

Copy link
Contributor Author

commented Sep 2, 2015

@MarioKnight Fixed.

lukecarbis pushed a commit that referenced this pull request Sep 2, 2015

Luke Carbis
Merge pull request #749 from xwp/bugfix/issue-733
Add a migration script for upgrading to Stream from 1.4.9

@lukecarbis lukecarbis merged commit 55d7679 into develop Sep 2, 2015

1 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@lukecarbis lukecarbis deleted the bugfix/issue-733 branch Sep 2, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.