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

Upgrade 2-4-stable to Solidus #1

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

Upgrade 2-4-stable to Solidus #1

wants to merge 14 commits into from

Conversation

Sinetheta
Copy link
Owner

The only notable changes were Spree::Config[:site_url] -> current_store.urland Spree::Config[:site_name] -> current_store.name.

These were uneeded gems just causing problems
We need this just to have access to a working test_app
These files are the entry points into Solidus for gem specific styles
and scripts. Without them we get templating errors.
Things have moved around a bit in solidus
Since this gem has no assets anyways I would rather remove this from the
rake task than add palceholder files.
These configs no longer work, but we have helpers to do the same.
This line would work in a normal app, but the specs are run in the
context of the dummy app, so Rails.root + spec is not going to find
anything.
We need these to survive spree's authentication check.
Although nothing requires authentication, we still need to help spree
boot up for a request spec.
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.

1 participant