search

metridoc-archives / metridoc-grails-core

all grails functionality for metridoc
0 stars 0 forks source link

Refactor and remove old views #96

Closed TApicella closed 11 years ago

TApicella commented 11 years ago

Ok here is a description of the organization

There are 3 tabs: Manage Access, General Settings, and LDAP Config

LDAP Config is a combination of ldapRole and ldapSettings. views/ldapRole contains the main views; views/ldapSettings contains only templates.

General Settings is a combination of status, config, and restart. All of the views are now collected in manageConfig. There is now a _config, a _restart, and a _status.

Manage Access is a combination of user, role, and manage report. All of the templates were moved into views/manageAccess. views/role and views/user each contain the edit and show views, since those are still specific to those controllers.

The only changes to the controllers was the deletion of unneeded list and index methods.

tbarker9 commented 11 years ago

The restart part of the ManageConfig no longer works. I suspect this has been the case since we migrated it to the format it is currently in. I have not used this feature since I added it, and happy to get rid of this. BUT, this worries me that other aspects of the application may not be working. Please comb through all the config stuff, add users , delete them, add roles, etc, and make sure everything works. And delete everything related to restart while you are at it. This means we don't need the codemirror javascript stuff at all.