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

Spree 1.x.x support + Banners Removed + More #8

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

Conversation

buddhi-desilva
Copy link

Hope I didn't desecrate the noble work you did with this gem :-) Did some major changes, minor tweaks and other updating work.

Changes

  • Spree 1.x.x Compatibility
  • Homepage products can be reordered + homepage products can be removed from homepage from the reorder screen
  • No more banners: Banners could work better (IMHO) as a separate gem
    for developers and store admins. (In addition to that banners were broken on spree 1.x.x and
    I didn't try fixing them either)

Planned features:
* Order of the homepage products

ToFix:
* Banners
* Banners may be best suited to be added as a separate gem
* Version bump (1.0.1)
CONT. FROM PREV UPDATE
* Banners removed: Banners could work better (IMHO) as a separate gem
for devs and store admins. (In addition to that banners were broken and
I didn't try fixing them either ;-)
* Spree 1.x.x support
* Homepage products can be reordered
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