FE for share groups with groups

changed milestone to %12.4

added Deliverable backend database databasereview pending devopsmanage groupauthentication and authorization [DEPRECATED] typefeature labels

removed database databasereview pending labels

added frontend label and removed backend label

added database databasereview pending labels

	6 Warnings
	This merge request is quite big (more than 617 lines changed), please consider splitting it into multiple merge requests.
	This merge request changed files with disabled eslint rules. Please consider fixing them.
	474fc2bb: This commit’s subject line is acceptable, but please try to reduce it to 50 characters.
	02123136: This commit’s subject line is acceptable, but please try to reduce it to 50 characters.
	767d72ac: This commit’s subject line is acceptable, but please try to reduce it to 50 characters.
	8c137b58: This commit’s subject line is acceptable, but please try to reduce it to 50 characters.

1 Message

CHANGELOG missing: If this merge request doesn’t need a CHANGELOG entry, feel free to ignore this message.

You can create one with:

bin/changelog -m 17139 "FE for share groups with groups"

If you want to create a changelog entry for GitLab EE, run the following instead:

bin/changelog --ee -m 17139 "FE for share groups with groups"

Note: Merge requests with ~backstage, ci-build, meta do not trigger this check.

Disabled eslint rules

The following files have disabled eslint rules. Please consider fixing them:

app/assets/javascripts/pages/groups/group_members/index/index.js

Run the following command for more details

node_modules/.bin/eslint --report-unused-disable-directives --no-inline-config \
  'app/assets/javascripts/pages/groups/group_members/index/index.js'

Reviewer roulette

Changes that require review have been detected! A merge request is normally reviewed by both a reviewer and a maintainer in its primary category (e.g. frontend or backend), and by a maintainer in all other categories.

To spread load more evenly across eligible reviewers, Danger has randomly picked a candidate for each review slot. Feel free to override this selection if you think someone else would be better-suited, or the chosen person is unavailable.

Once you've decided who will review this merge request, mention them as you normally would! Danger does not (yet?) automatically notify them for you.

Category	Reviewer	Maintainer
frontend	Dennis Tang (`@dennis`)	Natalia Tepluhina (`@ntepluhina`)
backend	Ethan Urie (`@eurie`)	Peter Leitzen (`@splattael`)
test Quality for `spec/features/*`	Désirée Chevalier (`@dchevalier2`)	No maintainer available
QA	Zeff Morgan (`@zeffmorgan`)	Walmyr Lima e Silva Filho (`@wlsf82`)

Generated by Danger

added 1225 commits

1da1ebbd...1ea8d32d - 1223 commits from branch master
ad476338 - Share groups with groups (BE)
7a4ce8a6 - FE for share groups with groups

Failed Job	This branch	Master	Differences in failures
`ee:kubernetes`	Job	Job	identical (timed out)
`ee:docker`	Job	Job	identical
`ee:update 2/5`	Job	Job	identical
`ee:update 1/5`	Job	Job	1 less failure in this branch
`ee:relative_url 2/5`	Job	Job	identical
`ee:instance 2/5`	Job	Job	identical
`ee:relative_url 1/5`	Job	Job	1 less failure in this branch
`ee:instance 1/5`	Job	Job	1 less failure in this branch

FE for share groups with groups

What does this MR do?

Screenshots

Does this MR meet the acceptance criteria?

Conformity

Performance and Testing

Merge request reports

Activity

Disabled eslint rules

Reviewer roulette