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

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

Open
timsayshey opened this issue Sep 5, 2012 · 4 comments
Open

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

timsayshey opened this issue Sep 5, 2012 · 4 comments

Comments

@timsayshey
Copy link

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
Copy link
Owner

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
Copy link
Author

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
Copy link
Owner

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
Copy link
Author

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

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

No branches or pull requests

2 participants