Rocket.Chat icon indicating copy to clipboard operation
Rocket.Chat copied to clipboard

feat: new moderation setting to enable trust roles (WM-74)

Open Dnouv opened this issue 2 years ago • 6 comments

Proposed changes (including videos or screenshots)

This PR introduces trust/reputation roles. To make it more customizable, there is a boolean setting, Trust Roles, once enabled it creates two new locked roles with varying permissions to the workspace; this is to be used by the RC Apps to provide different levels of services/privileges to the users, based on their trust/reputation roles. The Reputation App will manage the award/deduction of reputation points.

The last two columns are the new roles image

Issue(s)

Steps to test or reproduce

Further comments

JIRA: WM-74

Dnouv avatar May 29 '23 18:05 Dnouv

🦋 Changeset detected

Latest commit: 6a9a1a2c246fa243c5106d26035f8da4c3fd16db

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 28 packages
Name Type
@rocket.chat/core-services Minor
@rocket.chat/meteor Minor
@rocket.chat/account-service Patch
@rocket.chat/authorization-service Patch
@rocket.chat/ddp-streamer Patch
@rocket.chat/omnichannel-transcript Patch
@rocket.chat/presence-service Patch
@rocket.chat/queue-worker Patch
@rocket.chat/stream-hub-service Patch
@rocket.chat/omnichannel-services Patch
@rocket.chat/presence Patch
rocketchat-services Patch
@rocket.chat/core-typings Minor
@rocket.chat/rest-typings Minor
@rocket.chat/cron Patch
@rocket.chat/gazzodown Major
@rocket.chat/model-typings Patch
@rocket.chat/ui-contexts Major
@rocket.chat/api-client Patch
@rocket.chat/pdf-worker Patch
@rocket.chat/ddp-client Patch
@rocket.chat/fuselage-ui-kit Major
@rocket.chat/models Patch
@rocket.chat/ui-client Major
@rocket.chat/ui-video-conf Major
@rocket.chat/uikit-playground Patch
@rocket.chat/web-ui-registration Major
@rocket.chat/instance-status Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

changeset-bot[bot] avatar May 29 '23 18:05 changeset-bot[bot]

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: Project coverage is 64.20%. Comparing base (f164099) to head (6a9a1a2). :warning: Report is 4238 commits behind head on develop.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##           develop   #29393       +/-   ##
============================================
+ Coverage    47.14%   64.20%   +17.06%     
============================================
  Files          689       52      -637     
  Lines        13066     1341    -11725     
  Branches      2279      262     -2017     
============================================
- Hits          6160      861     -5299     
+ Misses        6569      426     -6143     
+ Partials       337       54      -283     
Flag Coverage Δ
e2e ?
unit 64.20% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov[bot] avatar May 29 '23 18:05 codecov[bot]

Btw - this is on hold afaik.

Please correct me if Im wrong @Dnouv

debdutdeb avatar Jul 05 '23 17:07 debdutdeb

Thank you, Hugo, for approving the changes.

Yes, this is currently in Product and UX discussion. So, converting it back into a draft.

Dnouv avatar Jul 05 '23 18:07 Dnouv

This PR currently has a merge conflict. Please resolve this and then re-add the ['stat: ready to merge', 'automerge'] label.

kodiakhq[bot] avatar Jul 21 '23 02:07 kodiakhq[bot]

Looks like this PR is not ready to merge, because of the following issues:

  • This PR has conflicts, please resolve them before merging
  • This PR is missing the 'stat: QA assured' label
  • This PR is not mergeable
  • This PR is missing the required milestone or project

Please fix the issues and try again

If you have any trouble, please check the PR guidelines

dionisio-bot[bot] avatar Oct 15 '24 18:10 dionisio-bot[bot]