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

DOC-4381: Add defintions for platform groups (#218) #222

Open
wants to merge 7 commits into
base: 5.2
Choose a base branch
from

Conversation

mzorai
Copy link
Contributor

@mzorai mzorai commented Apr 9, 2020

  • DOC-4381: Add definitions for platform groups
  • DOC-4381: Add details for membership types
  • DOC-4381: FIx typo
  • Update docs/Administration.rst
    Co-Authored-By: Ali HAMDI [email protected]
  • DOC-4381: Add details about spaces groups
    Co-authored-by: Ali HAMDI [email protected]

(cherry picked from commit 78c5360)

* DOC-4381: Add definitions for platform groups
* DOC-4381: Add details for membership types
* DOC-4381: FIx typo
* Update docs/Administration.rst
Co-Authored-By: Ali HAMDI <[email protected]>
* DOC-4381: Add details about spaces groups
Co-authored-by: Ali HAMDI <[email protected]>

(cherry picked from commit 78c5360)
Copy link
Member

@plamarque plamarque left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

suggested additions and precisions on groups and roles.
Also checked the list of images in the PR. many images modified i. the diff. Seems too much

docs/Administration.rst Outdated Show resolved Hide resolved

**Platform** helps to define roles in the platform, it contains 4 sub groups:
- Administration: contains users having administrative roles i.e. who are able to access the Administration menu |imagee354|.
- Guests: contains external users to the platform, they have access only to public content of the platform
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If I remember, no user is really assigned this group. It’s a membership automatically assigned to non authenticated users and thus, this technical group is mandatory to set permissions on this visible by non authenticated users only (such as pages and content)

**Platform** helps to define roles in the platform, it contains 4 sub groups:
- Administration: contains users having administrative roles i.e. who are able to access the Administration menu |imagee354|.
- Guests: contains external users to the platform, they have access only to public content of the platform
- Users: contains all the platform users able to connect to the platform
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IMO it’s important to explain that this is also a technical group mandatory for authe ticated isers. Users that are not members of isers geoup will see an http 403 (forbidden) when they try to connect

docs/Administration.rst Outdated Show resolved Hide resolved
docs/Administration.rst Outdated Show resolved Hide resolved

In eXo Platform, the most used memships are `manager`, `member` and `*`.

- The `manager` role to define users with managing privileges
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it gives space management peivileges but I also thnk manager membership type has something to do with navigation management in the prtal and maybe administrative privileges in community management (to ne verified, I may be wrong)

In eXo Platform, the most used memships are `manager`, `member` and `*`.

- The `manager` role to define users with managing privileges
- The `member` role to define normal users in a group
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe worth adding : such as members of a space

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

members of a group have access to the corresponding group site

Remaining roles could be used when you create your
:ref:`own publication lifecycle <PLFDevGuide.DevelopingContent.PublicationLifecycle.CreatingPublicationLifecycle>`.

.. note:: For :ref:`Spaces groups <spacesGroups>`, a user belonging to a space could be either a manager or a member.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think news is adding more roles in spaces around news publication. @samehaidi ?

docs/Administration.rst Outdated Show resolved Hide resolved
mzorai and others added 4 commits April 9, 2020 21:34
Co-Authored-By: Patrice Lamarque <[email protected]>
Co-Authored-By: Patrice Lamarque <[email protected]>
Co-Authored-By: Patrice Lamarque <[email protected]>
Co-Authored-By: Patrice Lamarque <[email protected]>
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

Successfully merging this pull request may close these issues.

2 participants