Skip to content

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
    • Help
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
GitLab Community Edition
GitLab Community Edition
  • Project
    • Project
    • Details
    • Activity
    • Releases
    • Cycle Analytics
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Charts
    • Locked Files
  • Issues 14,493
    • Issues 14,493
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge Requests 812
    • Merge Requests 812
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
    • Charts
  • Registry
    • Registry
  • Snippets
    • Snippets
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • GitLab.org
  • GitLab Community EditionGitLab Community Edition
  • Merge Requests
  • !3323

Merged
Opened Mar 20, 2016 by Douwe Maan@DouweM13 of 13 tasks completed13/13 tasks
  • Report abuse
Report abuse

Add group visibility level

Supersedes !3051 (closed)

Closes #12658 (closed)

To do

  • Update Admin group edit form to set visibility level
  • See if visible_to_user and public_to_user can be combined and/or renamed
  • Don't use current_application_settings in migration
  • Make sure are tested adequately:
    • JoinedGroupsFinder
    • GroupProjectsFinder
    • GroupsFinder
    • PersonalProjectsFinder
    • Groups::UpdateService
    • Groups::CreateService
    • ProjectsFinder
    • SnippetsFinder
    • Project#visibility_level_allowed?
  • Discussion 66
  • Commits 25
  • Pipelines 11
  • Changes 101
Robert Speicher
Assignee
Robert Speicher @rspeicher
Assign to
8.6
Milestone
8.6
Assign milestone
Time tracking
0
Labels
None
Assign labels
  • View project labels
Reference: gitlab-org/gitlab-ce!3323

Cherry-pick this merge request

Switch branch
Cancel
A new branch will be created in your fork and a new merge request will be started.