Skip to content
Snippets Groups Projects

Access tokens: Migrate HelpIcon to shared component

Merged Sascha Eggenberger requested to merge access-token-help-icon-migration into master
All threads resolved!

What does this MR do and why?

Access tokens: Migrate HelpIcon to shared 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/flightjs/Flight/-/settings/access_tokens
  2. http://gdk.test:3000/groups/flightjs/-/settings/access_tokens
  3. http://gdk.test:3000/-/user_settings/personal_access_tokens

Related to #499695 (closed), #507991 (closed)

Edited by Sascha Eggenberger

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
  • added pipelinetier-3 label and removed pipelinetier-1 label

  • Rahul Chanila approved this merge request

    approved this merge request

  • Bundle size analysis [beta]

    This compares changes in bundle size for entry points between the commits 700b9eca and cc42fe21

    :sparkles: Special assets

    Entrypoint / Name Size before Size after Diff Diff in percent
    average 4.41 MB 4.41 MB - 0.0 %
    mainChunk 3.28 MB 3.28 MB - 0.0 %

    Note: We do not have exact data for 700b9eca. So we have used data from: 63f92bea.
    The intended commit has no webpack pipeline, so we chose the last commit with one before it.

    Please look at the full report for more details


    Read more about how this report works.

    Generated by :no_entry_sign: Danger

  • E2E Test Result Summary

    allure-report-publisher generated test report!

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

    expand test summary
    +------------------------------------------------------------------+
    |                          suites summary                          |
    +-------------+--------+--------+---------+-------+-------+--------+
    |             | passed | failed | skipped | flaky | total | result |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Plan        | 82     | 0      | 8       | 0     | 90    | ✅     |
    | Verify      | 48     | 0      | 17      | 0     | 65    | ✅     |
    | Govern      | 80     | 0      | 12      | 0     | 92    | ✅     |
    | Create      | 135    | 0      | 20      | 0     | 155   | ✅     |
    | Release     | 5      | 0      | 1       | 0     | 6     | ✅     |
    | Monitor     | 8      | 0      | 12      | 0     | 20    | ✅     |
    | Package     | 25     | 0      | 13      | 0     | 38    | ✅     |
    | Fulfillment | 2      | 0      | 7       | 0     | 9     | ✅     |
    | Data Stores | 33     | 0      | 10      | 0     | 43    | ✅     |
    | Manage      | 1      | 0      | 9       | 0     | 10    | ✅     |
    | Configure   | 0      | 0      | 3       | 0     | 3     | ➖     |
    | Ai-powered  | 0      | 0      | 2       | 0     | 2     | ➖     |
    | Secure      | 4      | 0      | 3       | 0     | 7     | ✅     |
    | Growth      | 0      | 0      | 2       | 0     | 2     | ➖     |
    | Analytics   | 2      | 0      | 0       | 0     | 2     | ✅     |
    | ModelOps    | 0      | 0      | 1       | 0     | 1     | ➖     |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Total       | 425    | 0      | 120     | 0     | 545   | ✅     |
    +-------------+--------+--------+---------+-------+-------+--------+

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

    expand test summary
    +------------------------------------------------------------------+
    |                          suites summary                          |
    +-------------+--------+--------+---------+-------+-------+--------+
    |             | passed | failed | skipped | flaky | total | result |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Data Stores | 33     | 0      | 10      | 0     | 43    | ✅     |
    | Create      | 140    | 0      | 19      | 1     | 159   | ✅     |
    | Monitor     | 8      | 0      | 12      | 0     | 20    | ✅     |
    | Verify      | 49     | 0      | 16      | 0     | 65    | ✅     |
    | Release     | 5      | 0      | 1       | 0     | 6     | ✅     |
    | Package     | 30     | 0      | 14      | 0     | 44    | ✅     |
    | Plan        | 86     | 0      | 8       | 0     | 94    | ✅     |
    | Govern      | 84     | 0      | 10      | 0     | 94    | ✅     |
    | Secure      | 2      | 0      | 5       | 0     | 7     | ✅     |
    | Manage      | 1      | 0      | 9       | 0     | 10    | ✅     |
    | ModelOps    | 0      | 0      | 1       | 0     | 1     | ➖     |
    | Analytics   | 2      | 0      | 0       | 0     | 2     | ✅     |
    | Fulfillment | 2      | 0      | 7       | 0     | 9     | ✅     |
    | Configure   | 0      | 0      | 3       | 0     | 3     | ➖     |
    | Ai-powered  | 0      | 0      | 2       | 0     | 2     | ➖     |
    | Growth      | 0      | 0      | 2       | 0     | 2     | ➖     |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Total       | 442    | 0      | 119     | 1     | 561   | ✅     |
    +-------------+--------+--------+---------+-------+-------+--------+
  • Rahul Chanila resolved all threads

    resolved all threads

  • Rahul Chanila enabled automatic add to merge train when checks pass

    enabled automatic add to merge train when checks pass

  • Rahul Chanila requested review from @eduardosanz

    requested review from @eduardosanz

  • Eduardo Sanz García approved this merge request

    approved this merge request

  • Eduardo Sanz García resolved all threads

    resolved all threads

  • Rahul Chanila removed this merge request from the merge train because the pipeline did not succeed. Learn more.

    removed this merge request from the merge train because the pipeline did not succeed. Learn more.

  • Eduardo Sanz García enabled automatic add to merge train when checks pass

    enabled automatic add to merge train when checks pass

  • Eduardo Sanz García resolved all threads

    resolved all threads

  • mentioned in commit 9b4e5924

  • added workflowstaging label and removed workflowcanary label

  • Please register or sign in to reply
    Loading