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 initial Spree migration article #2640

Merged
merged 1 commit into from
Apr 17, 2018

Conversation

benjaminwil
Copy link
Contributor

This article walks the reader through a simple migrations from SpreeSpree 2.4 to Solidus 1.4. Then, it encourages readers to incrementally upgrade to newer versions of Solidus.

Both Spree 2.4 and Solidus 1.4 are pretty old now. It would be great to have an additional article that details a (more complicated) migration from Spree 3.x to Solidus 2.x. This would necessarily need to be simplified.

This is part a larger project to improve Solidus's documentation. See this gist with the high-level table of contents. Where and how this documentation will exist is still up for discussion.

Copy link
Contributor

@swcraig swcraig left a comment

Choose a reason for hiding this comment

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

Go Ben Go 🥇


If you use Rails's default asset pipeline, for example, we recommend that you
explicitly update the versions of the `sass-rails` gem in your `Gemfile`, as
well as explicitly setting the versions for the dependencies `sprocket` and
Copy link
Contributor

Choose a reason for hiding this comment

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

sprockets

```

At this point, the update should run successfully. If it does not, check your
`Gemfile` for other gems that require specific updated to their dependencies.
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this should be "require specific updates".

### Manage depreciation warnings

After upgrading your Spree 2.4 application to Solidus 1.4, you may see some
deprecation warnings. The amount of depreciation warnings depends on how many
Copy link
Contributor

Choose a reason for hiding this comment

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

"The amount of deprecation warnings"


After upgrading your Spree 2.4 application to Solidus 1.4, you may see some
deprecation warnings. The amount of depreciation warnings depends on how many
Spree features you use that are being depreciated in Solidus, as well as
Copy link
Contributor

Choose a reason for hiding this comment

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

I think "deprecated" instead of "depreciated".

Spree features you use that are being depreciated in Solidus, as well as
additional gems your application uses.

If you plan to upgrade to Solidus 2.0, you need to deal with the depreciated
Copy link
Contributor

Choose a reason for hiding this comment

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

I think "deprecated" instead of "depreciated".

break existing functionality in your application code.

Use the notes in [Solidus's changelog][changelog] to help you upgrade to each
minor version of Solidus gracefully.
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice word choice 🥇

## Migrating from Spree 3.0 or newer

Solidus is a fork of Spree 2.4. After Spree 2.4, the Spree and Solidus code
bases start to diverge. For example, Spree 3.0 introduces is a Bootstrap-based
Copy link
Contributor

Choose a reason for hiding this comment

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

"Spree 3.0 introduces is a"

frontend and backend which is significantly different than Solidus's frontend
and backend.

time and preparation than other migrations. If you run Spree 3.0 and want to
Copy link
Contributor

Choose a reason for hiding this comment

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

I think the start of this sentence got chopped somewhere.

@benjaminwil benjaminwil force-pushed the migration_documentation branch 2 times, most recently from 9733724 to 96a280d Compare March 20, 2018 23:18
This article walks the reader through a simple migrations from Spree
Spree 2.4 to Solidus 1.4. Then, it encourages readers to incrementally
upgrade to newer versions of Solidus.

Both Spree 2.4 and Solidus 1.4 are pretty old now. It would be great to
have an additional article that details a (more complicated) migration
from Spree 3.x to Solidus 2.x. This would necessarily need to be
simplified.
@gmacdougall gmacdougall merged commit a863473 into solidusio:master Apr 17, 2018
@benjaminwil benjaminwil deleted the migration_documentation branch April 30, 2018 18:28
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.

None yet

4 participants