[Discover] fix: Clean up sync URL subscription in Discover plugin topNav #9316

Maosaic · 2025-01-31T18:47:05Z

Description

Properly clean up sync URL subscription in Discover plugin topNav component.

Changes this PR contains:

Create useSyncQueryStateWithUrl hook which automatically cancel sync URL subscription.
Invoke useSyncQueryStateWithUrl from topNav component, so the URL subscription will be properly clean up when user navigate away from Discover page.
~~Rewrite SavedObjectSaveModal as functional component.~~

Issues Resolved

fixes #9309

Screenshot

Before

Screen.Recording.2025-01-29.at.2.56.41.PM.mp4

After

Screen.Recording.2025-01-31.at.10.43.38.AM.mp4

Testing the changes

Changelog

fix: Clean up sync URL subscription in Discover plugin topNav

Check List

All tests pass
- yarn test:jest
- yarn test:jest_integration
New functionality includes testing.
New functionality has been documented.
Update CHANGELOG.md
Commits are signed per the DCO using --signoff

codecov · 2025-01-31T19:02:36Z

Codecov Report

Attention: Patch coverage is 73.33333% with 4 lines in your changes missing coverage. Please review.

Project coverage is 61.71%. Comparing base (ec6d7df) to head (2a0bfad).
Report is 14 commits behind head on main.

Files with missing lines	Patch %	Lines
...plication/components/top_nav/get_top_nav_links.tsx	0.00%	3 Missing ⚠️
...lic/application/view_components/canvas/top_nav.tsx	0.00%	1 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #9316   +/-   ##
=======================================
  Coverage   61.70%   61.71%           
=======================================
  Files        3816     3817    +1     
  Lines       91829    91841   +12     
  Branches    14543    14545    +2     
=======================================
+ Hits        56666    56677   +11     
- Misses      31507    31508    +1     
  Partials     3656     3656

Flag	Coverage Δ
Linux_1	`28.98% <6.66%> (-0.01%)`	⬇️
Linux_2	`56.46% <ø> (ø)`
Linux_3	`39.19% <73.33%> (+0.01%)`	⬆️
Linux_4	`28.90% <6.66%> (-0.01%)`	⬇️
Windows_1	`29.00% <6.66%> (-0.01%)`	⬇️
Windows_2	`56.41% <ø> (ø)`
Windows_3	`39.19% <73.33%> (+0.01%)`	⬆️
Windows_4	`28.90% <6.66%> (-0.01%)`	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

src/plugins/data/public/query/state_sync/use_sync_state_with_url.test.ts

src/plugins/saved_objects/public/save_modal/saved_object_save_modal.tsx

Signed-off-by: Joey Liu <[email protected]>

kavilla · 2025-02-06T00:09:57Z

do you mind checking the ci failures? is the ci failing on main?

Maosaic · 2025-02-06T18:35:50Z

do you mind checking the ci failures? is the ci failing on main?

I see the same test error in other PR, I don't think that's related to my change

abbyhu2000 · 2025-02-12T19:34:59Z

@Maosaic one comment but not blocking.

I see we use useSyncQueryStateWithUrl to wrap syncQueryStateWithUrl for discover plugin.

Other plugins also uses syncQueryStateWithUrl
*dashboard

OpenSearch-Dashboards/src/plugins/dashboard/public/application/utils/create_dashboard_app_state.tsx

Line 131 in 4da33a1

const { stop: stopSyncingQueryServiceStateWithUrl } = syncQueryStateWithUrl(

OpenSearch-Dashboards/src/plugins/dashboard/public/application/components/dashboard_listing/dashboard_listing.tsx

Line 51 in 4da33a1

const { stop } = syncQueryStateWithUrl(query, osdUrlStateStorage);

*visbuilder

OpenSearch-Dashboards/src/plugins/vis_builder/public/application/app.tsx

Line 45 in 4da33a1

const { stop } = syncQueryStateWithUrl(query, osdUrlStateStorage);

*visualization

OpenSearch-Dashboards/src/plugins/visualize/public/application/app.tsx

Line 63 in 4da33a1

const { stop } = syncQueryStateWithUrl(query, osdUrlStateStorage);

should we also replace those with useSyncQueryStateWithUrl as well?

…Nav (#9316) * [Discover] fix: Clean up sync URL subscription in Discover plugin topNav Signed-off-by: Joey Liu <[email protected]> * Changeset file for PR #9316 created/updated * Update unit test Signed-off-by: Joey Liu <[email protected]> * Revert save modal change Signed-off-by: Joey Liu <[email protected]> --------- Signed-off-by: Joey Liu <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> (cherry picked from commit 8cc5f84) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

Maosaic · 2025-02-13T02:12:23Z

@Maosaic one comment but not blocking.

I see we use useSyncQueryStateWithUrl to wrap syncQueryStateWithUrl for discover plugin.

Other plugins also uses syncQueryStateWithUrl *dashboard

OpenSearch-Dashboards/src/plugins/dashboard/public/application/utils/create_dashboard_app_state.tsx

Line 131 in 4da33a1

const { stop: stopSyncingQueryServiceStateWithUrl } = syncQueryStateWithUrl(

OpenSearch-Dashboards/src/plugins/dashboard/public/application/components/dashboard_listing/dashboard_listing.tsx

Line 51 in 4da33a1

const { stop } = syncQueryStateWithUrl(query, osdUrlStateStorage);

*visbuilder

OpenSearch-Dashboards/src/plugins/vis_builder/public/application/app.tsx

Line 45 in 4da33a1

const { stop } = syncQueryStateWithUrl(query, osdUrlStateStorage);

*visualization

OpenSearch-Dashboards/src/plugins/visualize/public/application/app.tsx

Line 63 in 4da33a1

const { stop } = syncQueryStateWithUrl(query, osdUrlStateStorage);

should we also replace those with useSyncQueryStateWithUrl as well?

We can, but not necessary, those are manually calling stop

…Nav (#9316) (#9376) * [Discover] fix: Clean up sync URL subscription in Discover plugin topNav * Changeset file for PR #9316 created/updated * Update unit test * Revert save modal change --------- (cherry picked from commit 8cc5f84) Signed-off-by: Joey Liu <[email protected]> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>

…Nav (opensearch-project#9316) * [Discover] fix: Clean up sync URL subscription in Discover plugin topNav Signed-off-by: Joey Liu <[email protected]> * Changeset file for PR opensearch-project#9316 created/updated * Update unit test Signed-off-by: Joey Liu <[email protected]> * Revert save modal change Signed-off-by: Joey Liu <[email protected]> --------- Signed-off-by: Joey Liu <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]>

* add max recent queries tests Signed-off-by: Federico Silva <[email protected]> * refactor callback function Signed-off-by: Federico Silva <[email protected]> * add testid-42 tests Signed-off-by: Federico Silva <[email protected]> * add comments Signed-off-by: Federico Silva <[email protected]> * add spec to package.json Signed-off-by: Federico Silva <[email protected]> * fix testconfig fn name Signed-off-by: Federico Silva <[email protected]> * fix clipboard flakiness Signed-off-by: Federico Silva <[email protected]> * refactor test preparation Signed-off-by: Federico Silva <[email protected]> * fix package.json, yarn.locl and cypress.config.ts Signed-off-by: Federico Silva <[email protected]> * Changeset file for PR #9307 created/updated Signed-off-by: Federico Silva <[email protected]> * remove element.js references Signed-off-by: Federico Silva <[email protected]> * [Documentation] Add alternative Docker Development Environment Setup documentation. (#9362) * Add alternative Docker Development Environment Setup. Signed-off-by: Argus Li <[email protected]> * Changeset file for PR #9362 created/updated * Use relative link Signed-off-by: Argus Li <[email protected]> * Address Suchit's comments. Signed-off-by: Argus Li <[email protected]> --------- Signed-off-by: Argus Li <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * [TESTID-77] Index Patterns Caching functionality for Discover (#9331) * Create caching test. Signed-off-by: Argus Li <[email protected]> * Changeset file for PR #9331 created/updated * Create caching test. Signed-off-by: Argus Li <[email protected]> * Changeset file for PR #9331 created/updated * Refactor based on #9319 Signed-off-by: Argus Li <[email protected]> * Update package.json Signed-off-by: Suchit Sahoo <[email protected]> * Remove element.js Signed-off-by: Argus Li <[email protected]> --------- Signed-off-by: Argus Li <[email protected]> Signed-off-by: Suchit Sahoo <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Co-authored-by: Suchit Sahoo <[email protected]> Co-authored-by: Anan Zhuang <[email protected]> Signed-off-by: Federico Silva <[email protected]> * comment out tests with dependencies Signed-off-by: Federico Silva <[email protected]> * [Discover] fix: Clean up sync URL subscription in Discover plugin topNav (#9316) * [Discover] fix: Clean up sync URL subscription in Discover plugin topNav Signed-off-by: Joey Liu <[email protected]> * Changeset file for PR #9316 created/updated * Update unit test Signed-off-by: Joey Liu <[email protected]> * Revert save modal change Signed-off-by: Joey Liu <[email protected]> --------- Signed-off-by: Joey Liu <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * [TESTID-64,80,UI] Add cypress test for autocomplete feature (#9322) * [TESTID-64,80,UI] Add cypress test for autocomplete feature Add tests related to autocomplete. This will close all the issues listed here: https://github.com/opensearch-project/OpenSearch-Dashboards/issues/assigned/ananzh?q=is%3Aissue%20assignee%3Aananzh%20label%3A%22discover%20autocomplete%22%20 Signed-off-by: Anan <[email protected]> * Changeset file for PR #9322 created/updated --------- Signed-off-by: Anan <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * [Discover] chore: Update query editor loading UI (#9344) * [Discover] chore: Update query editor loading UI * Move progress bar position Signed-off-by: Joey Liu <[email protected]> * Update loading text font Signed-off-by: Joey Liu <[email protected]> * update unit test snapshot Signed-off-by: Joey Liu <[email protected]> * Update single line query editor Signed-off-by: Joey Liu <[email protected]> * Changeset file for PR #9344 created/updated * pull doc update Signed-off-by: Joey Liu <[email protected]> --------- Signed-off-by: Joey Liu <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * feat(vis_type_vega): support reading time field (#9152) * feat(vis_type_vega): support reading time field Signed-off-by: Yulong Ruan <[email protected]> * Changeset file for PR #9152 created/updated * fix time format to use moment.utc Signed-off-by: Yulong Ruan <[email protected]> --------- Signed-off-by: Yulong Ruan <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * [TESTID-17] Add tests for query testing and advanced settings (#9384) Issue Resolved: #8934 Signed-off-by: Anan <[email protected]> Signed-off-by: Federico Silva <[email protected]> --------- Signed-off-by: Federico Silva <[email protected]> Signed-off-by: Argus Li <[email protected]> Signed-off-by: Suchit Sahoo <[email protected]> Signed-off-by: Joey Liu <[email protected]> Signed-off-by: Anan <[email protected]> Signed-off-by: Yulong Ruan <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Co-authored-by: Argus Li <[email protected]> Co-authored-by: Suchit Sahoo <[email protected]> Co-authored-by: Anan Zhuang <[email protected]> Co-authored-by: Joey Liu <[email protected]> Co-authored-by: Yulong Ruan <[email protected]>

* Update actions/cache from v1 to v4 to address deprecation warning (#9366) * build: Update actions/cache from v1 to v4 to address deprecation warning Signed-off-by: Anan <[email protected]> * Changeset file for PR #9366 created/updated --------- Signed-off-by: Anan <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * Handle invalid geospatial request in region map (#8759) Signed-off-by: Junqiu Lei <[email protected]> Signed-off-by: Federico Silva <[email protected]> * Update DEVELOPER_GUIDE.md (#9368) When building the OpenSearch Dashboards artifact for Linux, the build may fail on certain distributions. This issue stems from the recent upgrade from node-sass to dart-sass. The new process uses an embeddable module from dart-sass to process Sass files, which relies on a platform-specific compiler. On Linux, this module depends on glibc. However, some distributions—like Alpine Linux—use musl libc instead, which is incompatible with this module. I'm updating the DEVELOPER_GUIDE documentation to help other developers choose the appropriate distro for Docker images and avoid the pitfalls I encountered. See the issue I created in Github explaining this reasoning: #9329 Also the forum where I posted the kind of error you see when building with a Linux Distribution that is incompatible with glibc: https://forum.opensearch.org/t/docker-a-error-error-worker-exitted-unexpectedly-with-code-1-last-message-bundleid-embeddable-type-running/23214 Signed-off-by: Luis Beltrán <[email protected]> Signed-off-by: Federico Silva <[email protected]> * create histogram tests Signed-off-by: Federico Silva <[email protected]> * update callback, refactor tests Signed-off-by: Federico Silva <[email protected]> * fix hooks Signed-off-by: Federico Silva <[email protected]> * add comments to code, refactor preconditions Signed-off-by: Federico Silva <[email protected]> * update hooks to beforeEach and afterEach for CI/CD Signed-off-by: Federico Silva <[email protected]> * Changeset file for PR #9290 created/updated Signed-off-by: Federico Silva <[email protected]> * add todos Signed-off-by: Federico Silva <[email protected]> * add to package.json Signed-off-by: Federico Silva <[email protected]> * remove elements.js references Signed-off-by: Federico Silva <[email protected]> * [Documentation] Add alternative Docker Development Environment Setup documentation. (#9362) * Add alternative Docker Development Environment Setup. Signed-off-by: Argus Li <[email protected]> * Changeset file for PR #9362 created/updated * Use relative link Signed-off-by: Argus Li <[email protected]> * Address Suchit's comments. Signed-off-by: Argus Li <[email protected]> --------- Signed-off-by: Argus Li <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * [TESTID-77] Index Patterns Caching functionality for Discover (#9331) * Create caching test. Signed-off-by: Argus Li <[email protected]> * Changeset file for PR #9331 created/updated * Create caching test. Signed-off-by: Argus Li <[email protected]> * Changeset file for PR #9331 created/updated * Refactor based on #9319 Signed-off-by: Argus Li <[email protected]> * Update package.json Signed-off-by: Suchit Sahoo <[email protected]> * Remove element.js Signed-off-by: Argus Li <[email protected]> --------- Signed-off-by: Argus Li <[email protected]> Signed-off-by: Suchit Sahoo <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Co-authored-by: Suchit Sahoo <[email protected]> Co-authored-by: Anan Zhuang <[email protected]> Signed-off-by: Federico Silva <[email protected]> * [Discover] fix: Clean up sync URL subscription in Discover plugin topNav (#9316) * [Discover] fix: Clean up sync URL subscription in Discover plugin topNav Signed-off-by: Joey Liu <[email protected]> * Changeset file for PR #9316 created/updated * Update unit test Signed-off-by: Joey Liu <[email protected]> * Revert save modal change Signed-off-by: Joey Liu <[email protected]> --------- Signed-off-by: Joey Liu <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * skip permutation for SQL Signed-off-by: Federico Silva <[email protected]> * add todo Signed-off-by: Federico Silva <[email protected]> * [TESTID-64,80,UI] Add cypress test for autocomplete feature (#9322) * [TESTID-64,80,UI] Add cypress test for autocomplete feature Add tests related to autocomplete. This will close all the issues listed here: https://github.com/opensearch-project/OpenSearch-Dashboards/issues/assigned/ananzh?q=is%3Aissue%20assignee%3Aananzh%20label%3A%22discover%20autocomplete%22%20 Signed-off-by: Anan <[email protected]> * Changeset file for PR #9322 created/updated --------- Signed-off-by: Anan <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * [Discover] chore: Update query editor loading UI (#9344) * [Discover] chore: Update query editor loading UI * Move progress bar position Signed-off-by: Joey Liu <[email protected]> * Update loading text font Signed-off-by: Joey Liu <[email protected]> * update unit test snapshot Signed-off-by: Joey Liu <[email protected]> * Update single line query editor Signed-off-by: Joey Liu <[email protected]> * Changeset file for PR #9344 created/updated * pull doc update Signed-off-by: Joey Liu <[email protected]> --------- Signed-off-by: Joey Liu <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * feat(vis_type_vega): support reading time field (#9152) * feat(vis_type_vega): support reading time field Signed-off-by: Yulong Ruan <[email protected]> * Changeset file for PR #9152 created/updated * fix time format to use moment.utc Signed-off-by: Yulong Ruan <[email protected]> --------- Signed-off-by: Yulong Ruan <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * [TESTID-17] Add tests for query testing and advanced settings (#9384) Issue Resolved: #8934 Signed-off-by: Anan <[email protected]> Signed-off-by: Federico Silva <[email protected]> * add to package.json Signed-off-by: Federico Silva <[email protected]> --------- Signed-off-by: Anan <[email protected]> Signed-off-by: Federico Silva <[email protected]> Signed-off-by: Junqiu Lei <[email protected]> Signed-off-by: Luis Beltrán <[email protected]> Signed-off-by: Argus Li <[email protected]> Signed-off-by: Suchit Sahoo <[email protected]> Signed-off-by: Joey Liu <[email protected]> Signed-off-by: Yulong Ruan <[email protected]> Signed-off-by: Anan Zhuang <[email protected]> Co-authored-by: Anan Zhuang <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Co-authored-by: Junqiu Lei <[email protected]> Co-authored-by: Luis Beltrán <[email protected]> Co-authored-by: Argus Li <[email protected]> Co-authored-by: Suchit Sahoo <[email protected]> Co-authored-by: Joey Liu <[email protected]> Co-authored-by: Yulong Ruan <[email protected]>

…Nav (opensearch-project#9316) * [Discover] fix: Clean up sync URL subscription in Discover plugin topNav Signed-off-by: Joey Liu <[email protected]> * Changeset file for PR opensearch-project#9316 created/updated * Update unit test Signed-off-by: Joey Liu <[email protected]> * Revert save modal change Signed-off-by: Joey Liu <[email protected]> --------- Signed-off-by: Joey Liu <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>

* add max recent queries tests Signed-off-by: Federico Silva <[email protected]> * refactor callback function Signed-off-by: Federico Silva <[email protected]> * add testid-42 tests Signed-off-by: Federico Silva <[email protected]> * add comments Signed-off-by: Federico Silva <[email protected]> * add spec to package.json Signed-off-by: Federico Silva <[email protected]> * fix testconfig fn name Signed-off-by: Federico Silva <[email protected]> * fix clipboard flakiness Signed-off-by: Federico Silva <[email protected]> * refactor test preparation Signed-off-by: Federico Silva <[email protected]> * fix package.json, yarn.locl and cypress.config.ts Signed-off-by: Federico Silva <[email protected]> * Changeset file for PR opensearch-project#9307 created/updated Signed-off-by: Federico Silva <[email protected]> * remove element.js references Signed-off-by: Federico Silva <[email protected]> * [Documentation] Add alternative Docker Development Environment Setup documentation. (opensearch-project#9362) * Add alternative Docker Development Environment Setup. Signed-off-by: Argus Li <[email protected]> * Changeset file for PR opensearch-project#9362 created/updated * Use relative link Signed-off-by: Argus Li <[email protected]> * Address Suchit's comments. Signed-off-by: Argus Li <[email protected]> --------- Signed-off-by: Argus Li <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * [TESTID-77] Index Patterns Caching functionality for Discover (opensearch-project#9331) * Create caching test. Signed-off-by: Argus Li <[email protected]> * Changeset file for PR opensearch-project#9331 created/updated * Create caching test. Signed-off-by: Argus Li <[email protected]> * Changeset file for PR opensearch-project#9331 created/updated * Refactor based on opensearch-project#9319 Signed-off-by: Argus Li <[email protected]> * Update package.json Signed-off-by: Suchit Sahoo <[email protected]> * Remove element.js Signed-off-by: Argus Li <[email protected]> --------- Signed-off-by: Argus Li <[email protected]> Signed-off-by: Suchit Sahoo <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Co-authored-by: Suchit Sahoo <[email protected]> Co-authored-by: Anan Zhuang <[email protected]> Signed-off-by: Federico Silva <[email protected]> * comment out tests with dependencies Signed-off-by: Federico Silva <[email protected]> * [Discover] fix: Clean up sync URL subscription in Discover plugin topNav (opensearch-project#9316) * [Discover] fix: Clean up sync URL subscription in Discover plugin topNav Signed-off-by: Joey Liu <[email protected]> * Changeset file for PR opensearch-project#9316 created/updated * Update unit test Signed-off-by: Joey Liu <[email protected]> * Revert save modal change Signed-off-by: Joey Liu <[email protected]> --------- Signed-off-by: Joey Liu <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * [TESTID-64,80,UI] Add cypress test for autocomplete feature (opensearch-project#9322) * [TESTID-64,80,UI] Add cypress test for autocomplete feature Add tests related to autocomplete. This will close all the issues listed here: https://github.com/opensearch-project/OpenSearch-Dashboards/issues/assigned/ananzh?q=is%3Aissue%20assignee%3Aananzh%20label%3A%22discover%20autocomplete%22%20 Signed-off-by: Anan <[email protected]> * Changeset file for PR opensearch-project#9322 created/updated --------- Signed-off-by: Anan <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * [Discover] chore: Update query editor loading UI (opensearch-project#9344) * [Discover] chore: Update query editor loading UI * Move progress bar position Signed-off-by: Joey Liu <[email protected]> * Update loading text font Signed-off-by: Joey Liu <[email protected]> * update unit test snapshot Signed-off-by: Joey Liu <[email protected]> * Update single line query editor Signed-off-by: Joey Liu <[email protected]> * Changeset file for PR opensearch-project#9344 created/updated * pull doc update Signed-off-by: Joey Liu <[email protected]> --------- Signed-off-by: Joey Liu <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * feat(vis_type_vega): support reading time field (opensearch-project#9152) * feat(vis_type_vega): support reading time field Signed-off-by: Yulong Ruan <[email protected]> * Changeset file for PR opensearch-project#9152 created/updated * fix time format to use moment.utc Signed-off-by: Yulong Ruan <[email protected]> --------- Signed-off-by: Yulong Ruan <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * [TESTID-17] Add tests for query testing and advanced settings (opensearch-project#9384) Issue Resolved: opensearch-project#8934 Signed-off-by: Anan <[email protected]> Signed-off-by: Federico Silva <[email protected]> --------- Signed-off-by: Federico Silva <[email protected]> Signed-off-by: Argus Li <[email protected]> Signed-off-by: Suchit Sahoo <[email protected]> Signed-off-by: Joey Liu <[email protected]> Signed-off-by: Anan <[email protected]> Signed-off-by: Yulong Ruan <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Co-authored-by: Argus Li <[email protected]> Co-authored-by: Suchit Sahoo <[email protected]> Co-authored-by: Anan Zhuang <[email protected]> Co-authored-by: Joey Liu <[email protected]> Co-authored-by: Yulong Ruan <[email protected]>

* Update actions/cache from v1 to v4 to address deprecation warning (opensearch-project#9366) * build: Update actions/cache from v1 to v4 to address deprecation warning Signed-off-by: Anan <[email protected]> * Changeset file for PR opensearch-project#9366 created/updated --------- Signed-off-by: Anan <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * Handle invalid geospatial request in region map (opensearch-project#8759) Signed-off-by: Junqiu Lei <[email protected]> Signed-off-by: Federico Silva <[email protected]> * Update DEVELOPER_GUIDE.md (opensearch-project#9368) When building the OpenSearch Dashboards artifact for Linux, the build may fail on certain distributions. This issue stems from the recent upgrade from node-sass to dart-sass. The new process uses an embeddable module from dart-sass to process Sass files, which relies on a platform-specific compiler. On Linux, this module depends on glibc. However, some distributions—like Alpine Linux—use musl libc instead, which is incompatible with this module. I'm updating the DEVELOPER_GUIDE documentation to help other developers choose the appropriate distro for Docker images and avoid the pitfalls I encountered. See the issue I created in Github explaining this reasoning: opensearch-project#9329 Also the forum where I posted the kind of error you see when building with a Linux Distribution that is incompatible with glibc: https://forum.opensearch.org/t/docker-a-error-error-worker-exitted-unexpectedly-with-code-1-last-message-bundleid-embeddable-type-running/23214 Signed-off-by: Luis Beltrán <[email protected]> Signed-off-by: Federico Silva <[email protected]> * create histogram tests Signed-off-by: Federico Silva <[email protected]> * update callback, refactor tests Signed-off-by: Federico Silva <[email protected]> * fix hooks Signed-off-by: Federico Silva <[email protected]> * add comments to code, refactor preconditions Signed-off-by: Federico Silva <[email protected]> * update hooks to beforeEach and afterEach for CI/CD Signed-off-by: Federico Silva <[email protected]> * Changeset file for PR opensearch-project#9290 created/updated Signed-off-by: Federico Silva <[email protected]> * add todos Signed-off-by: Federico Silva <[email protected]> * add to package.json Signed-off-by: Federico Silva <[email protected]> * remove elements.js references Signed-off-by: Federico Silva <[email protected]> * [Documentation] Add alternative Docker Development Environment Setup documentation. (opensearch-project#9362) * Add alternative Docker Development Environment Setup. Signed-off-by: Argus Li <[email protected]> * Changeset file for PR opensearch-project#9362 created/updated * Use relative link Signed-off-by: Argus Li <[email protected]> * Address Suchit's comments. Signed-off-by: Argus Li <[email protected]> --------- Signed-off-by: Argus Li <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * [TESTID-77] Index Patterns Caching functionality for Discover (opensearch-project#9331) * Create caching test. Signed-off-by: Argus Li <[email protected]> * Changeset file for PR opensearch-project#9331 created/updated * Create caching test. Signed-off-by: Argus Li <[email protected]> * Changeset file for PR opensearch-project#9331 created/updated * Refactor based on opensearch-project#9319 Signed-off-by: Argus Li <[email protected]> * Update package.json Signed-off-by: Suchit Sahoo <[email protected]> * Remove element.js Signed-off-by: Argus Li <[email protected]> --------- Signed-off-by: Argus Li <[email protected]> Signed-off-by: Suchit Sahoo <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Co-authored-by: Suchit Sahoo <[email protected]> Co-authored-by: Anan Zhuang <[email protected]> Signed-off-by: Federico Silva <[email protected]> * [Discover] fix: Clean up sync URL subscription in Discover plugin topNav (opensearch-project#9316) * [Discover] fix: Clean up sync URL subscription in Discover plugin topNav Signed-off-by: Joey Liu <[email protected]> * Changeset file for PR opensearch-project#9316 created/updated * Update unit test Signed-off-by: Joey Liu <[email protected]> * Revert save modal change Signed-off-by: Joey Liu <[email protected]> --------- Signed-off-by: Joey Liu <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * skip permutation for SQL Signed-off-by: Federico Silva <[email protected]> * add todo Signed-off-by: Federico Silva <[email protected]> * [TESTID-64,80,UI] Add cypress test for autocomplete feature (opensearch-project#9322) * [TESTID-64,80,UI] Add cypress test for autocomplete feature Add tests related to autocomplete. This will close all the issues listed here: https://github.com/opensearch-project/OpenSearch-Dashboards/issues/assigned/ananzh?q=is%3Aissue%20assignee%3Aananzh%20label%3A%22discover%20autocomplete%22%20 Signed-off-by: Anan <[email protected]> * Changeset file for PR opensearch-project#9322 created/updated --------- Signed-off-by: Anan <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * [Discover] chore: Update query editor loading UI (opensearch-project#9344) * [Discover] chore: Update query editor loading UI * Move progress bar position Signed-off-by: Joey Liu <[email protected]> * Update loading text font Signed-off-by: Joey Liu <[email protected]> * update unit test snapshot Signed-off-by: Joey Liu <[email protected]> * Update single line query editor Signed-off-by: Joey Liu <[email protected]> * Changeset file for PR opensearch-project#9344 created/updated * pull doc update Signed-off-by: Joey Liu <[email protected]> --------- Signed-off-by: Joey Liu <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * feat(vis_type_vega): support reading time field (opensearch-project#9152) * feat(vis_type_vega): support reading time field Signed-off-by: Yulong Ruan <[email protected]> * Changeset file for PR opensearch-project#9152 created/updated * fix time format to use moment.utc Signed-off-by: Yulong Ruan <[email protected]> --------- Signed-off-by: Yulong Ruan <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Signed-off-by: Federico Silva <[email protected]> * [TESTID-17] Add tests for query testing and advanced settings (opensearch-project#9384) Issue Resolved: opensearch-project#8934 Signed-off-by: Anan <[email protected]> Signed-off-by: Federico Silva <[email protected]> * add to package.json Signed-off-by: Federico Silva <[email protected]> --------- Signed-off-by: Anan <[email protected]> Signed-off-by: Federico Silva <[email protected]> Signed-off-by: Junqiu Lei <[email protected]> Signed-off-by: Luis Beltrán <[email protected]> Signed-off-by: Argus Li <[email protected]> Signed-off-by: Suchit Sahoo <[email protected]> Signed-off-by: Joey Liu <[email protected]> Signed-off-by: Yulong Ruan <[email protected]> Signed-off-by: Anan Zhuang <[email protected]> Co-authored-by: Anan Zhuang <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> Co-authored-by: Junqiu Lei <[email protected]> Co-authored-by: Luis Beltrán <[email protected]> Co-authored-by: Argus Li <[email protected]> Co-authored-by: Suchit Sahoo <[email protected]> Co-authored-by: Joey Liu <[email protected]> Co-authored-by: Yulong Ruan <[email protected]>

github-actions bot added the repeat-contributor label Jan 31, 2025

opensearch-changeset-bot bot added a commit to Maosaic/OpenSearch-Dashboards that referenced this pull request Jan 31, 2025

Changeset file for PR opensearch-project#9316 created/updated

9adae0f

ashwin-pc reviewed Feb 4, 2025

View reviewed changes

src/plugins/data/public/query/state_sync/use_sync_state_with_url.test.ts Show resolved Hide resolved

kavilla added backport 2.x discover for discover reinvent labels Feb 5, 2025

kavilla reviewed Feb 5, 2025

View reviewed changes

src/plugins/saved_objects/public/save_modal/saved_object_save_modal.tsx Outdated Show resolved Hide resolved

Maosaic and others added 4 commits February 5, 2025 21:58

[Discover] fix: Clean up sync URL subscription in Discover plugin topNav

60dd66e

Signed-off-by: Joey Liu <[email protected]>

Changeset file for PR opensearch-project#9316 created/updated

12c1957

Update unit test

39f4f2c

Signed-off-by: Joey Liu <[email protected]>

Revert save modal change

2a0bfad

Signed-off-by: Joey Liu <[email protected]>

Maosaic force-pushed the topnav branch from 4592edd to 2a0bfad Compare February 5, 2025 22:42

kavilla approved these changes Feb 6, 2025

View reviewed changes

kavilla assigned kavilla and abbyhu2000 Feb 6, 2025

abbyhu2000 approved these changes Feb 12, 2025

View reviewed changes

abbyhu2000 merged commit 8cc5f84 into opensearch-project:main Feb 12, 2025
69 of 71 checks passed

opensearch-trigger-bot bot mentioned this pull request Feb 12, 2025

[Backport 2.x] [Discover] fix: Clean up sync URL subscription in Discover plugin topNav #9376

Merged

Maosaic deleted the topnav branch February 13, 2025 02:12

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Discover] fix: Clean up sync URL subscription in Discover plugin topNav #9316

[Discover] fix: Clean up sync URL subscription in Discover plugin topNav #9316

Maosaic commented Jan 31, 2025 •

edited

Loading

codecov bot commented Jan 31, 2025 •

edited

Loading

kavilla commented Feb 6, 2025

Maosaic commented Feb 6, 2025

abbyhu2000 commented Feb 12, 2025

Maosaic commented Feb 13, 2025

[Discover] fix: Clean up sync URL subscription in Discover plugin topNav #9316

[Discover] fix: Clean up sync URL subscription in Discover plugin topNav #9316

Conversation

Maosaic commented Jan 31, 2025 • edited Loading

Description

Issues Resolved

Screenshot

Testing the changes

Changelog

Check List

codecov bot commented Jan 31, 2025 • edited Loading

Codecov Report

kavilla commented Feb 6, 2025

Maosaic commented Feb 6, 2025

abbyhu2000 commented Feb 12, 2025

Maosaic commented Feb 13, 2025

Maosaic commented Jan 31, 2025 •

edited

Loading

codecov bot commented Jan 31, 2025 •

edited

Loading