Skip to content
Snippets Groups Projects

Releases page: Crud migration

Merged Sascha Eggenberger requested to merge release-page-crud-migration into master
All threads resolved!

What does this MR do and why?

Releases page: Migrates to Crud component

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

How to set up and validate locally

  1. http://gdk.test:3000/gitlab-org/gitlab-shell/-/releases

Related to #452231 (closed)

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
  • Diana Zubova
  • Diana Zubova
  • Diana Zubova requested changes

    requested changes

  • Diana Zubova removed review request for @dzubova

    removed review request for @dzubova

  • added 1 commit

    Compare with previous version

  • requested review from @dzubova

  • Diana Zubova approved this merge request

    approved this merge request

  • Diana Zubova requested review from @jannik_lehmann and removed review request for @dzubova

    requested review from @jannik_lehmann and removed review request for @dzubova

  • Jannik Lehmann
  • Jannik Lehmann approved this merge request

    approved this merge request

  • Thanks for working on this @seggenberger and the great initial review @dzubova. This LGTM! There was nothing for me to find, except a non-blocking nitpick. For the sake of efficiency I will go ahead and merge this, if you want to address the nitpick please do so in a follow-up MR.

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

  • Jannik Lehmann resolved all threads

    resolved all threads

  • Jannik Lehmann enabled automatic add to merge train when checks pass

    enabled automatic add to merge train when checks pass

  • Jannik Lehmann added this merge request to the merge train at position 3

    added this merge request to the merge train at position 3

  • Jannik Lehmann mentioned in commit 82cdd3c2

    mentioned in commit 82cdd3c2

  • added workflowstaging label and removed workflowcanary label

  • Please register or sign in to reply
    Loading