Skip to content
Snippets Groups Projects

Fix invite a group on project wording and link

Merged Doug Stull requested to merge 393011-fix-doc-link-for-inivite-group into master

What does this MR do and why?

  • fix wording and link for inviting a group from a project as per #393011
  • fix margin bottom spacing under alert

Screenshots or screen recordings

source before after
project Screenshot_2023-04-03_at_4.42.16_PM Screenshot_2023-04-03_at_4.36.08_PM
group Screenshot_2023-04-03_at_4.42.34_PM Screenshot_2023-04-03_at_4.36.57_PM

How to set up and validate locally

bin/rails c

ApplicationSetting.first.update(dashboard_limit_enabled: true, dashboard_notification_limit: 3, dashboard_limit: 5, dashboard_enforcement_limit: 10)
Feature.enable(:free_user_cap)
Feature.enable(:free_user_cap_new_namespaces)
Feature.enable(:preview_free_user_cap)
  • Sign in as admin or another user. Create private group and private project inside.
  • Go to the group members page and click Invite a group.
  • Validate notification.
  • Go to the project members page and click Invite a group.
  • Validate notification.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #393011

Edited by Doug Stull

Merge request reports

Merged results pipeline #834000781 passed

Pipeline: GitLab

#834011976

    Pipeline: GitLab

    #834021375

      Merged results pipeline passed for db997cf3

      Test coverage 82.66% (15.17%) from 2 jobs

      Merged by Frédéric CapletteFrédéric Caplette 1 year ago (Apr 11, 2023 4:43pm UTC)

      Loading

      Pipeline #834091109 passed

      Pipeline passed for 11989411 on master

      Test coverage 67.20% (15.17%) from 2 jobs
      10 environments impacted.

      Activity

      Filter activity
      • Approvals
      • Assignees & reviewers
      • Comments (from bots)
      • Comments (from users)
      • Commits & branches
      • Edits
      • Labels
      • Lock status
      • Mentions
      • Merge request status
      • Tracking
      • Loading
      • Loading
      • Loading
      • Loading
      • Loading
      • Loading
      • Loading
      • Loading
      • Loading
      • Loading
      Please register or sign in to reply
      Loading