search

brightin / brightcontent

BrightContent, yet another cms / admin panel for Rails
brightin.nl
MIT License
12 stars 13 forks source link

Remove / replace inherited_resources #22

Closed jvdp closed 8 years ago

jvdp commented 10 years ago

The gem is deprecated: https://github.com/josevalim/inherited_resources#deprecation-notice. (I also find the extra levels of indirection pretty annoying, eg. if you want to find the implementation of the default actions.)

It should be easy to implement the basic actions generically ourselves (similar to the old DefaultAction module but perhaps a bit cleaner.) I think perhaps we should give up the support for nesting though, and it's something that you can (should?) setup easily per project.

Any thoughts?

jvdp commented 9 years ago

Some preliminary work here: https://github.com/jvdp/brightcontent/tree/remove-inherited-resources

jvdp commented 9 years ago

@stefanroex you tested my branch right? Any news?

stefanroex commented 9 years ago

Unfortunately, I haven't... :crying_cat_face:

jvdp commented 8 years ago

Closing in favor of #46