search

abitdodgy / cfw-usermanager

A user manager demo app for ColdFusion on Wheels
23 stars 11 forks source link

Had to move adminusers to make it work for 1.2-beta1 #16

Open timsayshey opened 12 years ago

timsayshey commented 12 years ago

Had to move /views/admin/adminusers to /views/adminusers to make it work for 1.2-beta1, not sure if you had to make a modification to the framework to make sub sub view folders work for the version you were using or not.

Would be nice to have it working with the latest version in the intended way :)

abitdodgy commented 12 years ago

Interesting. Did you try to play around with the renderPage() over ride?

https://github.com/abitdodgy/cfw-usermanager/blob/master/controllers/AdminUsers.cfc#L60

timsayshey commented 12 years ago

I missed that I'll have to take a look at it later on. I bet it doesn't work with 1.2-beta1 due to the fact that they are changing renderPage to renderView.

abitdodgy commented 12 years ago

Ahh, OK. I knowing nothing about 1.2. To be honest, I don't see a critical need to nest the admin views under a general admin folder. For organisation, it makes sense, but it does seem to throw a wrench in things. I'm thinking about flattening the structure. It shouldn't be bad if relavent folder names started with adminxxx.

Thoughts?

timsayshey commented 12 years ago

Yeah, I do like the idea of keeping it organized but simpler is better.