Dashboard visibility is defined by classes rather than by raw strings.

There are two reasons:
- we want to avoid identifying objects and drive logic by placing strings in code
- we want to distingush between visbility name which is stored in the datastore (for example, org_admin) and its verbose name which is shown to the users and must be human readable, i.e. it should not contain characters like _
4 files changed