Skip to content
Snippets Groups Projects

show/hide "make public" web option based on users' group membership

Merged Alexander Pace requested to merge hide-public-checkbox into master
All threads resolved!

if a user who isn't in access_managers tries to add the public tag, then they get a javascript error. this hides the checkbox if they don't have the group permission.

Merge request reports

Pipeline #625572 passed

Pipeline passed for 2efdc819 on hide-public-checkbox

Test coverage 72.00% (0.00%) from 1 job

Merged by Alexander PaceAlexander Pace 11 months ago (May 15, 2024 3:46pm UTC)

Loading

Pipeline #625966 passed

Pipeline passed for 285ed18c on master

Test coverage 72.00% (0.00%) from 1 job

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • added 1 commit

    • d1fd8b0f - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • added 1 commit

    • 41ba3fad - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • added 1 commit

    • 71116794 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • added 1 commit

    • 99e46aec - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Alexander Pace resolved all threads

    resolved all threads

  • added 1 commit

    • 2efdc819 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Daniel Wysocki approved this merge request

    approved this merge request

  • Tested on dev1 again, still checks out.

  • Alexander Pace mentioned in merge request !223 (merged)

    mentioned in merge request !223 (merged)

  • Please register or sign in to reply
    Loading