Skip to content

Badges settings: Migrate GlCard to CRUD

Sascha Eggenberger requested to merge badges-migrate-card-to-crud into master

What does this MR do and why?

This is a follow-up to Migrate badges card to crud component (!161229 - merged)

  • Migrates the form used in Crud
  • Simplifies code
  • Fixes a bug in Crud where the action still shows when the form is opened while it shouldn't
  • Changes the delete action to a default button (adds consistency with other implementations)
  • Adds tooltips to icon-only action buttons (Edit/Delete)

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 after
form_before form_after
tooltip_2
tooltip_1

How to set up and validate locally

  1. http://gdk.test:3000/groups/flightjs/-/edit
  2. http://gdk.test:3000/flightjs/Flight/edit

Related to #452231 (closed)

Edited by Sascha Eggenberger

Merge request reports