Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Backport main] Add new test suite and workflow for left navigation menu #1710

Merged

Conversation

FriedhelmWS
Copy link
Contributor

@FriedhelmWS FriedhelmWS commented Feb 12, 2025

Description

Manually backport 947aaa0 from #1707

Issues Resolved

N/A

Check List

  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

…search-project#1707)

* Add new test suite and workflow for left navigation menu

Signed-off-by: Owen Wang <[email protected]>

* Fix missing config for disable workspace

Signed-off-by: Owen Wang <[email protected]>

* Address comments by adding additional assertion to navigation menu

Signed-off-by: Owen Wang <[email protected]>

* Fix lint issue in dashboards test cases
Signed-off-by: Owen Wang <[email protected]>

---------

Signed-off-by: Owen Wang <[email protected]>
(cherry picked from commit 947aaa0)
@FriedhelmWS FriedhelmWS changed the title Backport/backport 1707 to main [Backport main] Add new test suite and workflow for left navigation menu Feb 12, 2025
@FriedhelmWS FriedhelmWS marked this pull request as ready for review February 12, 2025 10:17
@raintygao raintygao merged commit c207a75 into opensearch-project:main Feb 17, 2025
44 of 71 checks passed
@@ -6,6 +6,10 @@
/// <reference types="cypress" />

import { setTimeFilter, TRACE_ID } from '../../../utils/constants';
import customParseFormat from 'dayjs/plugin/customParseFormat';
import dayjs from 'dayjs';
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need to backport all the changes in this file? Seems dayjs not been used in test cases below.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants