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

Appland v2.9.5 #8

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

Appland v2.9.5 #8

wants to merge 24 commits into from

Conversation

jaredKreppein
Copy link

No description provided.

kennyadsl and others added 24 commits July 16, 2019 15:00
Avoid using navbar variables and general variables color, about Tabs.
Dedicated variables has been added.
We are already doing that for core preferences (plain Spree::Config).

This commit also allows to use stub_spree_preferences into stores and
extensions that define their own configuration classes, like
Spree::Auth::Config in solidus_auth_devise for example.
it's already the default value
Noticed that the custom sass `@function very-light` doesn't work
properly and this fact is generating display issues, both in tables
with actions, and in components such as Taxonomies.
Sprockets 4 uses manifest files[1], and the default manifest.js
links `app/assets/images`. To prevent it from crashing due to the
directory not being present, the easiest solution is to create an
empty `app/assets/images` directory.

[1]: https://github.com/rails/sprockets/blob/master/UPGRADING.md#manifestjs
The method was only "unused" within the Solidus codebase, but extensions
and apps were relying on it.

This reverts commit d82fb56.
It mixes different levels of configuration and just forwards to its
class level counterpart.
…oval-and-add-a-deprecation

Revert method removal and replace it with a deprecation
Was removed as part of solidusio#3081,
but not being private it's better to keep it around with a deprecation
for a while.
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.

7 participants