Skip to content
Snippets Groups Projects

Update reviewer status icons to outlined from solid

Merged Annabel Dunstone Gray requested to merge mr-sidebar-state into master
All threads resolved!

What does this MR do and why?

Update reviewer status icons in merge request sidebar

Why?

Screenshots or screen recordings

Before After
Screenshot_2024-02-29_at_11.44.04_AM Screenshot_2024-02-29_at_12.52.01_PM

How to set up and validate locally

Go to merge request, review MR with three states (approve, request changes, comment)

Links

Edited by Annabel Dunstone Gray

Merge request reports

Merged results pipeline #1205774762 passed

Pipeline: Ruby 3.1 as-if-foss pipeline

#1205778054

    Pipeline: E2E GDK

    #1205778806

      Pipeline: GitLab

      #1205785448

        +1

        Merged results pipeline passed for 0bfbb438

        Test coverage 81.52% (16.56%) from 2 jobs

        Merged by Jacques ErasmusJacques Erasmus 11 months ago (Mar 8, 2024 10:07am UTC)

        Merge details

        Pipeline #1206129787 failed

        Pipeline failed for 40fe881f on master

        Test coverage 64.77% (16.56%) from 2 jobs
        6 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
        Please register or sign in to reply
        Loading