Upgraded hook targets to deface overrides

citrus / spree_wholesale

A wholesale extension for Spree Commerce

http://rubygems.org/gems/spree_wholesale

BSD 3-Clause "New" or "Revised" License

36 stars 36 forks source link

Upgraded hook targets to deface overrides #3

Closed CameronCarroll closed 13 years ago

CameronCarroll commented 13 years ago

Hi. The hooks weren't working perfectly for me in edge spree and rails, so I decided to move them over to deface overrides. Most still target hooks, except for the admin menu tab which was giving trouble. (It's targeted on the div#store-menu ul instead.

Spent some time trying to make sure everything is still in place, and it looks good to me.

Thanks,

Sanarothe

CameronCarroll commented 13 years ago

Huh. Looks like somebody else already branched out and did this same thing, and he used the exact things Deface spat out as deprecation warnings. Should have checked that before I began... oh well. First contribution ever and it's extraneous. :)

I'll clean it up a little bit and add some documentation to make my commit better than his.

citrus commented 13 years ago

Thanks for your help!! I've merged your changes and added you to the list of contributors :)

Keep it up!