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

Removed logic for migration script #10081

Merged
merged 1 commit into from
Oct 30, 2018
Merged

Conversation

kirrg001
Copy link
Contributor

no issue

  • 2.0/6-replace-fixture-posts.js
  • we've reconsidered replacing the fixture posts
  • this PR removes the logic from this script

no issue

- we've reconsidered replacing the fixture posts
- this PR removes the logic from this script
@kirrg001
Copy link
Contributor Author

@gargol Could you please test that the migration from 1.25.5 to this branch works as expected for MySQL + SQlite3? Thanks 👍

@naz
Copy link
Contributor

naz commented Oct 29, 2018

@kirrg001 what's the reason this migration has to be removed?

@naz
Copy link
Contributor

naz commented Oct 29, 2018

Tested following scenarios on [email protected] & [email protected]:

  • setting up instance version @1.25.5 and migrating to 2.x (fixture posts from 1.x are still present)
  • setting up a new instance at 2.x (posts from 2.x are being added)

It's good to go! 🏃‍♀️

@allouis allouis merged commit 781f552 into TryGhost:master Oct 30, 2018
allouis added a commit that referenced this pull request Oct 30, 2018
* master:
  Version bump to 2.4.0
  Updated Ghost-Admin to 2.4.0
  Handled error from express-session middleware (#10084)
  Removed user_id constraint when upserting session (#10085)
  Added Node v10 Support (#10058)
  Removed logic for migration script (2.0/6-replace-fixture-posts.js) (#10081)
  Fixed sanitization issue in subscribers
  Parsed nconf env values (#10077)
  🐛 Fixed missing filename when exporting subscribers csv
  🐛 Fixed pagination for subscribers
  🐛 Fixed cardWidth being lost on 2.0 imports (#10068)
  Corrected 'Content-Length' header by using Buffer.byteLength (#10055)
  Fixed mail api usage of the notifcations api
  Refactored request unit tests to return promises (#10045)
  Refactored spam prevention tests to use promises (#10036)
  Enforced unix line endings (#9871)
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