Skip to content
Snippets Groups Projects

Support Group Avatar Removal in REST API

All threads resolved!

This change set is a follow up of !96381 (merged) and adds support to remove a Group Avatar via REST API. Similar to what I've implemented for the Project API in !92604 (merged).

The main driver for this change is the Terraform provider where is a highly requested feature to be able to manage group avatars.

Edited by See GitLab account @timofurrer

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • removed review request for @msedlakjakubowski

  • Allen Cook removed review request for @acook.gitlab

    removed review request for @acook.gitlab

  • 🤖 GitLab Bot 🤖 added Leading Organization label and removed 1 deleted label

    added Leading Organization label and removed 1 deleted label

  • requested review from @msedlakjakubowski

  • Marcin Sedlak-Jakubowski approved this merge request

    approved this merge request

  • added twfinished label and removed twdoing label

  • removed review request for @msedlakjakubowski

  • requested review from @carlad-gl

  • Carla Drago approved this merge request

    approved this merge request

  • Carla Drago removed review request for @carlad-gl

    removed review request for @carlad-gl

  • Allen Cook approved this merge request

    approved this merge request

  • Allen Cook requested review from @acook.gitlab

    requested review from @acook.gitlab

  • 1 Message
    :book: This merge request adds or changes documentation files. A review from the Technical Writing team before you merge is recommended. Reviews can happen after you merge.

    Documentation review

    The following files require a review from a technical writer:

    • doc/api/groups.md

    The review does not need to block merging this merge request. See the:

    Reviewer roulette

    Changes that require review have been detected!

    Please refer to the table below for assigning reviewers and maintainers suggested by Danger in the specified category:

    Category Reviewer Maintainer
    backend Brian Williams (@bwill) (UTC-5) Alex Pooley (@alexpooley) (UTC+8)

    To spread load more evenly across eligible reviewers, Danger has picked a candidate for each review slot, based on their timezone. Feel free to override these selections if you think someone else would be better-suited or use the GitLab Review Workload Dashboard to find other available reviewers.

    To read more on how to use the reviewer roulette, please take a look at the Engineering workflow and code review guidelines. Please consider assigning a reviewer or maintainer who is a domain expert in the area of the merge request.

    Once you've decided who will review this merge request, assign them as a reviewer! Danger does not automatically notify them for you.

    If needed, you can retry the :repeat: danger-review job that generated this comment.

    Generated by :no_entry_sign: Danger

  • Allen Cook resolved all threads

    resolved all threads

  • Allen Cook enabled an automatic merge when the pipeline for 1b7e0523 succeeds

    enabled an automatic merge when the pipeline for 1b7e0523 succeeds

  • merged

  • @tuxtimo, how was your code review experience with this merge request? Please tell us how we can continue to iterate and improve:

    1. Leave a :thumbsup: or a :thumbsdown: on this comment to describe your experience.
    2. Create a new comment starting with @gitlab-bot feedback below, and leave any additional feedback you have for us in the comment.

    Have five minutes? Take our survey to give us even more feedback on how GitLab can improve the contributor experience.

    Thanks for your help! :heart:

    This message was generated automatically. You're welcome to improve it.

  • Allen Cook mentioned in commit 0609c5e6

    mentioned in commit 0609c5e6

  • added workflowstaging label and removed workflowcanary label

  • Please register or sign in to reply
    Loading