Skip to content

Leave project/group from access granted email

Luke Bennett requested to merge member-access-granted-leave-email-fe into master

What does this MR do?

CE: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/27892

Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/59421

Remove yourself from a group or project with one click

Note, the 2nd last commit is a cherry-pick of https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/11864 so I didn't have to write code twice.

  • Add leave link to access_granted email

  • Handle leave in query string by clicking the leave link automagically

    • If no leave link, flash error

Screenshot_2019-04-30_at_09.03.18

Screenshot_2019-04-30_at_09.09.14

Screenshot_2019-04-30_at_09.03.41

Screenshot_2019-04-30_at_09.04.06

Screenshot_2019-04-30_at_08.55.10

Screenshot_2019-04-30_at_08.53.07

Follow-up issues

https://gitlab.com/gitlab-org/gitlab-ce/issues/61169

https://gitlab.com/gitlab-org/gitlab-ce/issues/61170

Does this MR meet the acceptance criteria?

Conformity

Performance and testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Luke Bennett

Merge request reports

Loading