Skip to content
Snippets Groups Projects

Change importing alert to warning

Merged Luke Duncalfe requested to merge ld-importing-alert-warning into master

What does this MR do and why?

This MR changes the Import in progress alert from danger to warning. See this Slack thread (internal, good for 90 days).

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
image image

How to set up and validate locally

  1. Comment out this line.
  2. Visit a project page

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
  • E2E Test Result Summary

    allure-report-publisher generated test report!

    e2e-test-on-gdk: :white_check_mark: test report for 3115a982

    expand test summary
    +------------------------------------------------------------------+
    |                          suites summary                          |
    +-------------+--------+--------+---------+-------+-------+--------+
    |             | passed | failed | skipped | flaky | total | result |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Manage      | 0      | 0      | 1       | 0     | 1     | ➖     |
    | Create      | 8      | 0      | 3       | 0     | 11    | ✅     |
    | Govern      | 3      | 0      | 0       | 0     | 3     | ✅     |
    | Package     | 0      | 0      | 1       | 0     | 1     | ➖     |
    | Monitor     | 4      | 0      | 0       | 0     | 4     | ✅     |
    | Plan        | 4      | 0      | 0       | 0     | 4     | ✅     |
    | Data Stores | 2      | 0      | 0       | 0     | 2     | ✅     |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Total       | 21     | 0      | 5       | 0     | 26    | ✅     |
    +-------------+--------+--------+---------+-------+-------+--------+

    e2e-package-and-test: :white_check_mark: test report for 3115a982

    expand test summary
    +------------------------------------------------------------------+
    |                          suites summary                          |
    +-------------+--------+--------+---------+-------+-------+--------+
    |             | passed | failed | skipped | flaky | total | result |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Manage      | 40     | 0      | 10      | 7     | 50    | ✅     |
    | Create      | 144    | 0      | 39      | 2     | 183   | ✅     |
    | Govern      | 6      | 0      | 0       | 0     | 6     | ✅     |
    | Plan        | 8      | 0      | 0       | 0     | 8     | ✅     |
    | Package     | 0      | 0      | 2       | 0     | 2     | ➖     |
    | Data Stores | 4      | 0      | 0       | 0     | 4     | ✅     |
    | Monitor     | 8      | 0      | 0       | 0     | 8     | ✅     |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Total       | 210    | 0      | 51      | 9     | 261   | ✅     |
    +-------------+--------+--------+---------+-------+-------+--------+
    Edited by Ghost User
  • Luke Duncalfe added 1 commit

    added 1 commit

    Compare with previous version

  • Justin Ho Tuan Duong resolved all threads

    resolved all threads

  • Justin Ho Tuan Duong enabled an automatic merge when all merge checks for 3115a982 pass

    enabled an automatic merge when all merge checks for 3115a982 pass

  • Justin Ho Tuan Duong approved this merge request

    approved this merge request

  • mentioned in commit 3e716408

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading