Skip to content

Resolve GlAvatar size in MR approvals footer

Jiaan Louw requested to merge 356452-fix-gl-avatar-size-in-approvals-footer into master

What does this MR do and why?

Updates the avatar size from 20 to 24 to resolve #356452 (closed) as part of the UserAvatarList migration to GlAvatar &7767 (closed).

Screenshots or screen recordings

before + feature flag disabled before + feature flag enabled after + feature flag enabled
gdk.test_3000_h5bp_html5-boilerplate<span data-escaped-char>_</span>-<span data-escaped-char>_</span>merge_requests_1 gdk.test_3000_h5bp_html5-boilerplate<span data-escaped-char>_</span>-merge_requests_1__1 gdk.test_3000_h5bp_html5-boilerplate_-merge_requests_1__7

The summary is being updated in !83747 (merged).

How to set up and validate locally

  1. Enable the feature flag by running: echo "Feature.enable(:gl_avatar_for_all_user_avatars)" | rails c
  2. View a merge request eligible approvers at: [GDK_HOST]/h5bp/html5-boilerplate/-/merge_requests/1
  3. If there are no eligible approvers, then edit the MR and add a MR approval rule for users or a group.

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 #356452 (closed)

Edited by Jiaan Louw

Merge request reports