Minor updates #8

Merged
merged 10 commits into from Nov 6, 2013

Conversation

Projects
None yet
2 participants
Contributor

nebirhos commented Oct 10, 2013

Ciao! I forked your gem and made some little enhancements:

  • Added a Search box for Banners in Admin
  • Refactored Helper and Configuration class
  • Some code cleanup

I'm testing a feature to attach the banner to a Taxon, if you find it useful I could make a pull request for that too.

Also, I created a 2-0-stable branch to keep master for Spree 2.1, I think you should do this in your repo.

damianogiacomello added a commit that referenced this pull request Nov 6, 2013

@damianogiacomello damianogiacomello merged commit 79b0bcf into damianogiacomello:master Nov 6, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment