Uploaded image for project: 'JBoss BPMS Platform'
  1. JBoss BPMS Platform
  2. RHBPMS-3643

Document - Roles for web UI should follow some naming convention

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Done
    • Icon: Critical Critical
    • None
    • 6.0.0
    • Documentation

      +++ This bug was initially created as a clone of Bug #1002522 +++

      Description of problem:
      Currently there are two roles that can be used when adding user that needs access to the Business Central. Role 'kie-user' for "normal" users and role 'ADMIN' that has the capability to create/clone repos, etc. Its bit weird that one of the roles is in lower case and second in upper case.

      We should follow some naming convention, e.g if we have 'kie-user', what about using 'kie-admin' instead of 'ADMIN'? Or maybe some different names, but I think we should be consistent in naming things.

      Note: Similarly the dashbuilder (BAM) needs role 'user', what about changing that also to 'kie-user'? Or could that cause issues?

      Version-Release number of selected component (if applicable):
      6.0.0-ER2

      — Additional comment from David Gutierrez on 2013-09-13 10:13:04 EDT —

      From the BPMS perspective the jBPM Dashboard (BAM) module defines two roles: kie-user and admin. Currently, there are two additional issues opened with regards on this.

      https://bugzilla.redhat.com/show_bug.cgi?id=1007729
      https://bugzilla.redhat.com/show_bug.cgi?id=1005981

      We all should align and agree on a unified role convention.

      — Additional comment from Catherine Robson on 2013-09-16 08:20:04 EDT —

      Refer to https://bugzilla.redhat.com/show_bug.cgi?id=1005981.

      This notes that there should be 5 roles. Suggested naming convention from UX:

      admin, analyst, developer, user, viewer

      — Additional comment from Prakash Aradhya on 2013-09-16 22:03:32 EDT —

      Guys, lets not expose users to cryptic kie-*** namings. Please follow the list of 5 users that are identified. Catherine has a link above.
      Please use the same role names both in the UI as well as documentation etc..

      — Additional comment from on 2013-09-23 05:23:50 EDT —

      kie-user=>analyst, ADMIN=>admin

      — Additional comment from David Gutierrez on 2013-09-23 05:31:25 EDT —

      The solution is:

      • To rename the current roles to 'analyst' and 'admin' (no caps and no 'kie' prefix). That will align the current roles definition with what is being proposed by PM and can be achieved in the short term.
      • Continue working on adding the additional roles requested (viewer, developer, user) in order to close this BZ. The changes are not trivial since they require some non-trivial modifications like enhancing the UF's authorization subsystem. So the deadline for that can't be established at this moment.

      — Additional comment from on 2013-09-24 04:26:28 EDT —

      kie-wb-common
      -------------
      (master) http://github.com/droolsjbpm/kie-wb-common/commit/a0af8aa9a

      (6.0.x) http://github.com/droolsjbpm/kie-wb-common/commit/90cceccc6

      drools-wb
      ---------
      (master) http://github.com/droolsjbpm/drools-wb/commit/731488bd8

      (6.0.x) http://github.com/droolsjbpm/drools-wb/commit/630b8d7c7

      kie-wb-distributions
      --------------------
      (master) https://github.com/droolsjbpm/kie-wb-distributions/commit/c8e2e0
      http://github.com/droolsjbpm/kie-wb-distributions/commit/a4a5f584b

      (6.0.x) https://github.com/droolsjbpm/kie-wb-distributions/commit/9afa6f
      http://github.com/droolsjbpm/kie-wb-distributions/commit/aca2940f5

      — Additional comment from on 2013-09-24 04:29:00 EDT —

      Set this BZ to MODIFIED as the remaining questions about Roles is captured under https://bugzilla.redhat.com/show_bug.cgi?id=1005981. At the moment roles "admin" and "analyst" have been unified.

            rhn-support-vigoyal Vikram Goyal (Inactive)
            rrajasek@redhat.com Rajesh Rajasekaran
            Marek Baluch Marek Baluch
            Marek Baluch Marek Baluch
            brms-docs brms-docs (Inactive), Catherine Weeks, David Gutierrez, Edson Tirelli, paradhya2, Petr Široký, Rajesh Rajasekaran, Vikram Goyal (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated: