Whereas PAGS previously had both the concept of a "name" of a PAGS group and a "key" for that group (which looks `fname`-ish), this code seems to collapse these concepts into just a user-facing "Name".  That's deliberate?  We're sure no one was relying on the ability to multiply implement groups that appear to have the same "Name", using different implementations in different contexts?  Which trick seems like a bad idea.  But just the same, the change to what is modeled about a PAGS group is deliberate?

Cf. [PAGSGroupStoreConfig.xml](https://github.com/Jasig/uPortal/blob/74bb37d2354001d817e1cee247276c1f329f813e/uportal-war/src/main/resources/properties/groups/PAGSGroupStoreConfig.xml#L136)

```
 <group>
    <group-key>respondr_theme_users</group-key>
    <group-name>Respondr Theme Users</group-name>
```