Skip to content
Snippets Groups Projects

Remove the `/vue` URL suffix on the new todos dashboard

Merged Paul Gascou-Vaillancourt requested to merge 501269-vue-todos-app-remove-url-suffix into master

What does this MR do and why?

Remove the /vue URL suffix on the new todos dashboard

The old and the new todos dashboards are now accessible at the same URL (dashboard/todos), but we render one or the other depending on the todos_vue_application feature flag's state.

The /vue URL will redirect to the base URL until we remove the feature flag.

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.

How to set up and validate locally

  1. Disable the todos_vue_application feature flag, then navigate to http://gdk.test:3000/dashboard/todos.
  2. Enable the todos_vue_application feature flag, then navigate to http://gdk.test:3000/dashboard/todos.

Related to #501269 (closed)

Edited by Paul Gascou-Vaillancourt

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
  • requested review from @nisong and @ohoral

  • Olena Horal-Koretska approved this merge request

    approved this merge request

  • Nwanna Isong approved this merge request

    approved this merge request

  • Nwanna Isong removed review request for @nisong

    removed review request for @nisong

  • requested review from @alipniagov

  • Aleksei Lipniagov approved this merge request

    approved this merge request

  • Aleksei Lipniagov enabled automatic add to merge train when checks pass

    enabled automatic add to merge train when checks pass

  • assigned to @leipert

  • E2E Test Result Summary

    allure-report-publisher generated test report!

    e2e-test-on-gdk: :white_check_mark: test report for 86b0b896

    expand test summary
    +------------------------------------------------------------------+
    |                          suites summary                          |
    +-------------+--------+--------+---------+-------+-------+--------+
    |             | passed | failed | skipped | flaky | total | result |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Data Stores | 33     | 0      | 1       | 0     | 34    | ✅     |
    | Create      | 129    | 0      | 22      | 0     | 151   | ✅     |
    | Verify      | 43     | 0      | 2       | 0     | 45    | ✅     |
    | Monitor     | 8      | 0      | 0       | 0     | 8     | ✅     |
    | Fulfillment | 2      | 0      | 0       | 0     | 2     | ✅     |
    | Plan        | 76     | 0      | 0       | 0     | 76    | ✅     |
    | Package     | 24     | 0      | 11      | 0     | 35    | ✅     |
    | Govern      | 75     | 0      | 3       | 0     | 78    | ✅     |
    | Release     | 5      | 0      | 0       | 0     | 5     | ✅     |
    | Secure      | 4      | 0      | 0       | 0     | 4     | ✅     |
    | Manage      | 1      | 0      | 1       | 0     | 2     | ✅     |
    | Analytics   | 2      | 0      | 0       | 0     | 2     | ✅     |
    | Ai-powered  | 0      | 0      | 1       | 0     | 1     | ➖     |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Total       | 402    | 0      | 41      | 0     | 443   | ✅     |
    +-------------+--------+--------+---------+-------+-------+--------+

    e2e-test-on-cng: :white_check_mark: test report for 86b0b896

    expand test summary
    +------------------------------------------------------------------+
    |                          suites summary                          |
    +-------------+--------+--------+---------+-------+-------+--------+
    |             | passed | failed | skipped | flaky | total | result |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Data Stores | 33     | 0      | 10      | 0     | 43    | ✅     |
    | Create      | 140    | 0      | 20      | 0     | 160   | ✅     |
    | Verify      | 49     | 0      | 16      | 0     | 65    | ✅     |
    | Govern      | 84     | 0      | 9       | 1     | 93    | ✅     |
    | Monitor     | 8      | 0      | 12      | 0     | 20    | ✅     |
    | Plan        | 86     | 0      | 8       | 0     | 94    | ✅     |
    | Secure      | 2      | 0      | 5       | 0     | 7     | ✅     |
    | Configure   | 0      | 0      | 3       | 0     | 3     | ➖     |
    | Fulfillment | 2      | 0      | 7       | 1     | 9     | ✅     |
    | Analytics   | 2      | 0      | 0       | 1     | 2     | ✅     |
    | Package     | 24     | 0      | 14      | 0     | 38    | ✅     |
    | Release     | 5      | 0      | 1       | 0     | 6     | ✅     |
    | Ai-powered  | 0      | 0      | 2       | 0     | 2     | ➖     |
    | Manage      | 1      | 0      | 9       | 0     | 10    | ✅     |
    | ModelOps    | 0      | 0      | 1       | 0     | 1     | ➖     |
    | Growth      | 0      | 0      | 2       | 0     | 2     | ➖     |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Total       | 436    | 0      | 119     | 3     | 555   | ✅     |
    +-------------+--------+--------+---------+-------+-------+--------+
  • Aleksei Lipniagov removed review request for @alipniagov

    removed review request for @alipniagov

  • changed milestone to %17.7

  • Lukas Eipert aborted automatic add to merge train because the source branch was updated. Learn more.

    aborted automatic add to merge train because the source branch was updated. Learn more.

  • Lukas Eipert added 4027 commits

    added 4027 commits

    Compare with previous version

  • Lukas Eipert reset approvals from @alipniagov and @ohoral by pushing to the branch

    reset approvals from @alipniagov and @ohoral by pushing to the branch

  • Lukas Eipert added 1 commit

    added 1 commit

    • 0b4ccc79 - Stub todos_vue_application in fixture generation

    Compare with previous version

  • Lukas Eipert requested review from @alipniagov

    requested review from @alipniagov

  • Lukas Eipert requested review from @ohoral

    requested review from @ohoral

  • mentioned in issue #483073 (closed)

  • Aleksei Lipniagov approved this merge request

    approved this merge request

  • Lukas Eipert resolved all threads

    resolved all threads

  • Lukas Eipert enabled automatic add to merge train when checks pass

    enabled automatic add to merge train when checks pass

  • Olena Horal-Koretska approved this merge request

    approved this merge request

  • Thomas Hutterer aborted automatic add to merge train because the source branch was updated. Learn more.

    aborted automatic add to merge train because the source branch was updated. Learn more.

  • added 1 commit

    • 86b0b896 - Update route in feature specs

    Compare with previous version

  • Thomas Hutterer reset approvals from @alipniagov and @ohoral by pushing to the branch

    reset approvals from @alipniagov and @ohoral by pushing to the branch

  • Thomas Hutterer resolved all threads

    resolved all threads

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