Skip to content
Snippets Groups Projects

Polish the instance overview cards

Merged Austin Regnery requested to merge aregnery/admin-overview-polish into master
All threads resolved!

What does this MR do and why?

The admin dashboard overview is the main landing page for Administrators. The Instance overview has been untouched for years and needed some UI polish 🧡

This change namely does three functional things:

  1. Combines the [ projects/users/groups ] stat cards with the list cards sitting farther below. This ties the content together
  2. Removes a decent number of ? links. Often these would point to documentation for configuration, so instead they are presented if there is not a direct link to adjust the configuration of the feature.
  3. Move the direct links to settings pages to a configure button :gear:

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

Before After
Before.png After - 5.png

How to set up and validate locally

  1. Open the Admin Area
  2. Check the links and various cards
Edited by Austin Regnery

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
  • Austin Regnery added 3 commits

    added 3 commits

    • 0315cdc9 - Fix spec to look for badge by testid
    • be20250a - Fix test for stat card
    • bc90d8d5 - Move up to date badge to footer

    Compare with previous version

  • Austin Regnery changed the description

    changed the description

  • Austin Regnery resolved all threads

    resolved all threads

  • added 1 commit

    Compare with previous version

  • Austin Regnery changed the description

    changed the description

  • Austin Regnery changed title from Polish the admin overview cards to Polish the instance overview cards

    changed title from Polish the admin overview cards to Polish the instance overview cards

  • Austin Regnery changed the description

    changed the description

  • Austin Regnery added 612 commits

    added 612 commits

    Compare with previous version

  • Austin Regnery resolved all threads

    resolved all threads

  • added 1 commit

    Compare with previous version

  • requested review from @panoskanell

  • Austin Regnery requested review from @vvempati

    requested review from @vvempati

  • Vamsi Vempati
  • Vamsi Vempati
  • Sascha Eggenberger approved this merge request

    approved this merge request

  • added pipelinetier-2 label and removed pipelinetier-1 label

  • Before you set this MR to auto-merge

    This merge request will progress on pipeline tiers until it reaches the last tier: pipelinetier-3. We will trigger a new pipeline for each transition to a higher tier.

    Before you set this MR to auto-merge, please check the following:

    • You are the last maintainer of this merge request
    • The latest pipeline for this merge request is pipelinetier-3 (You can find which tier it is in the pipeline name)
    • This pipeline is recent enough (created in the last 8 hours)

    If all the criteria above apply, please set auto-merge for this merge request.

    See pipeline tiers and merging a merge request for more details.

  • E2E Test Result Summary

    allure-report-publisher generated test report!

    e2e-test-on-gdk: :white_check_mark: test report for be8e1c45

    expand test summary
    +-------------------------------------------------------------+
    |                       suites summary                        |
    +--------+--------+--------+---------+-------+-------+--------+
    |        | passed | failed | skipped | flaky | total | result |
    +--------+--------+--------+---------+-------+-------+--------+
    | Govern | 1      | 0      | 0       | 0     | 1     | ✅     |
    +--------+--------+--------+---------+-------+-------+--------+
    | Total  | 1      | 0      | 0       | 0     | 1     | ✅     |
    +--------+--------+--------+---------+-------+-------+--------+

    e2e-test-on-cng: :white_check_mark: test report for be8e1c45

    expand test summary
    +------------------------------------------------------------------+
    |                          suites summary                          |
    +-------------+--------+--------+---------+-------+-------+--------+
    |             | passed | failed | skipped | flaky | total | result |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Create      | 139    | 0      | 20      | 15    | 159   | ✅     |
    | Manage      | 1      | 0      | 9       | 0     | 10    | ✅     |
    | Verify      | 50     | 0      | 15      | 10    | 65    | ✅     |
    | Monitor     | 8      | 0      | 12      | 0     | 20    | ✅     |
    | Plan        | 86     | 0      | 8       | 10    | 94    | ✅     |
    | Data Stores | 33     | 0      | 10      | 0     | 43    | ✅     |
    | Govern      | 78     | 0      | 11      | 8     | 89    | ✅     |
    | Fulfillment | 2      | 0      | 7       | 0     | 9     | ✅     |
    | Release     | 5      | 0      | 1       | 0     | 6     | ✅     |
    | Package     | 24     | 0      | 14      | 0     | 38    | ✅     |
    | Analytics   | 2      | 0      | 0       | 0     | 2     | ✅     |
    | Secure      | 4      | 0      | 2       | 1     | 6     | ✅     |
    | Configure   | 0      | 0      | 3       | 0     | 3     | ➖     |
    | Growth      | 0      | 0      | 2       | 0     | 2     | ➖     |
    | Ai-powered  | 0      | 0      | 2       | 0     | 2     | ➖     |
    | ModelOps    | 0      | 0      | 1       | 0     | 1     | ➖     |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Total       | 432    | 0      | 117     | 44    | 549   | ✅     |
    +-------------+--------+--------+---------+-------+-------+--------+
  • Vamsi Vempati removed review request for @vvempati

    removed review request for @vvempati

  • Panos Kanellidis removed review request for @panoskanell

    removed review request for @panoskanell

  • Austin Regnery added 682 commits

    added 682 commits

    Compare with previous version

  • Austin Regnery added 49 commits

    added 49 commits

    Compare with previous version

  • Austin Regnery added 61 commits

    added 61 commits

    Compare with previous version

  • requested review from @ccharnolevsky

  • Panos Kanellidis approved this merge request

    approved this merge request

  • requested review from @jprovaznik

  • Jan Provaznik approved this merge request

    approved this merge request

  • Jan Provaznik removed review request for @jprovaznik

    removed review request for @jprovaznik

  • Chen Charnolevsky requested review from @anastasia.khom and removed review request for @ccharnolevsky

    requested review from @anastasia.khom and removed review request for @ccharnolevsky

  • Austin Regnery added 417 commits

    added 417 commits

    Compare with previous version

  • requested changes

  • Austin Regnery changed milestone to %17.6

    changed milestone to %17.6

  • Austin Regnery added 1335 commits

    added 1335 commits

    Compare with previous version

  • added 1 commit

    • 001d981e - Add aria-label to configure buttons

    Compare with previous version

  • Austin Regnery reset approvals from @jprovaznik by pushing to the branch

    reset approvals from @jprovaznik by pushing to the branch

  • added workflowin review label and removed workflowin dev label

  • Anastasia Khomchenko approved this merge request

    approved this merge request

  • requested review from @psimyn

  • Simon Knox
  • Simon Knox
  • Simon Knox
  • Simon Knox
  • Simon Knox
  • Simon Knox
  • Simon Knox
  • Simon Knox requested changes

    requested changes

  • Simon Knox
  • Austin Regnery added 3 commits

    added 3 commits

    Compare with previous version

  • Simon Knox resolved all threads

    resolved all threads

  • Simon Knox approved this merge request

    approved this merge request

  • Simon Knox enabled automatic add to merge train when checks pass

    enabled automatic add to merge train when checks pass

  • merged

  • Simon Knox mentioned in commit 75c685d4

    mentioned in commit 75c685d4

  • added workflowstaging label and removed workflowcanary label

  • Please register or sign in to reply
    Loading