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

[Manual backport 2.x][Deangular] OSD without angular (#5086) #5177

Merged

Conversation

ashwin-pc
Copy link
Member

@ashwin-pc ashwin-pc commented Oct 3, 2023

Description

Manula backport of #5086

Issues Resolved

Check List

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

* Basic working version of OSD without angular (opensearch-project#4982)

Signed-off-by: Ashwin P Chandran <[email protected]>

* [Data Explorer][Discover 2.0] Implement saved search embeddable (opensearch-project#5081)

* [Data Explorer][Discover 2.0] Implement saved search embeddable

Issue Resolve
opensearch-project#4233

---------

Signed-off-by: ananzh <[email protected]>

* Deangular: removes angular paginate, cleanup data plugin and tilemap (opensearch-project#5060)

Signed-off-by: Ashwin P Chandran <[email protected]>

* [Data Explorer] Delete legacy discover selenium tests (opensearch-project#5088)

* delete legacy discover functional tests

Signed-off-by: abbyhu2000 <[email protected]>

* remove discover folder in config file

Signed-off-by: abbyhu2000 <[email protected]>

---------

Signed-off-by: abbyhu2000 <[email protected]>

* [Data Explorer][Discover][Function Test] Fix ciGroup4 (opensearch-project#5099)

* add data-render-complete and data-shared-item for test convenience
* change docTableFieldCount to dataGridTableCellCount to fetch data grid cell
* add methods in data grid service to reuse current massive selenium tests
* update tests

Issue Resolve
opensearch-project#5098
opensearch-project#5058

Signed-off-by: Anan Zhuang <[email protected]>

* [Data Explorer][Discover][Functional Test] fix ciGroup 8 (opensearch-project#5104)

Update index pattern fetch method in discover page due to changing to EuiComboBox.

Issue Resolve
opensearch-project#5103

Signed-off-by: Anan Zhuang <[email protected]>

* Fix cigroup 1 with dataGridTable helper functions (opensearch-project#5105)

* fix1

Signed-off-by: abbyhu2000 <[email protected]>

* fix 1

Signed-off-by: abbyhu2000 <[email protected]>

* fix anchor test

Signed-off-by: abbyhu2000 <[email protected]>

* fix cigroup1

Signed-off-by: abbyhu2000 <[email protected]>

* address comments

Signed-off-by: abbyhu2000 <[email protected]>

---------

Signed-off-by: abbyhu2000 <[email protected]>

* [Data Explorer] Fix functional cigroup 3 for save search on dashboard filtering (opensearch-project#5111)

* fix cigroup3

Signed-off-by: abbyhu2000 <[email protected]>

* fix error

Signed-off-by: abbyhu2000 <[email protected]>

---------

Signed-off-by: abbyhu2000 <[email protected]>

* add data-shared-item tag (opensearch-project#5119)

Signed-off-by: abbyhu2000 <[email protected]>

* Modifying html tests to utilize jsx and fixing tests to include all pluginPaths (opensearch-project#5080)

Signed-off-by: Zashary Maskus-Lavin <[email protected]>

* fix cigroup6 (opensearch-project#5125)

Signed-off-by: abbyhu2000 <[email protected]>

* [Discover] Search on load fix (opensearch-project#5112)

* fixes search on load

Signed-off-by: Ashwin P Chandran <[email protected]>

* Fixes search after initial page load

Signed-off-by: Ashwin P Chandran <[email protected]>

---------

Signed-off-by: Ashwin P Chandran <[email protected]>

* [Data Explorer][Discover][Functional Test] Fix ciGroup7 (opensearch-project#5131)

* [Data Explorer][Discover][Functional Test] Fix ciGroup7

Issue Resolve
opensearch-project#5129

Signed-off-by: Anan Zhuang <[email protected]>

* restore _discover_navigation

Signed-off-by: Anan Zhuang <[email protected]>

---------

Signed-off-by: Anan Zhuang <[email protected]>

* Deangular: removes angular from notify and utils of opendashboards legacy (opensearch-project#5085)

* Basic working version of OSD without angular (opensearch-project#4982)

Signed-off-by: Ashwin P Chandran <[email protected]>

* deangularized notify and utils folders in the opendashboards legacy directory

Signed-off-by: Cameron Reed <[email protected]>

---------

Signed-off-by: Ashwin P Chandran <[email protected]>
Signed-off-by: Cameron Reed <[email protected]>
Co-authored-by: Ashwin P Chandran <[email protected]>
Co-authored-by: Cameron Reed <[email protected]>

* [Discover] Misc fixes (opensearch-project#5114)

* Canvas scrolling fix, remove notice and improve flyout focus

Signed-off-by: Ashwin P Chandran <[email protected]>

* Hide action icons unless hovered or focussed on

Signed-off-by: Ashwin P Chandran <[email protected]>

* followup fixes from opensearch-project#5085

Signed-off-by: Ashwin P Chandran <[email protected]>

---------

Signed-off-by: Ashwin P Chandran <[email protected]>

---------

Signed-off-by: Ashwin P Chandran <[email protected]>
Signed-off-by: ananzh <[email protected]>
Signed-off-by: abbyhu2000 <[email protected]>
Signed-off-by: Anan Zhuang <[email protected]>
Signed-off-by: Zashary Maskus-Lavin <[email protected]>
Signed-off-by: Cameron Reed <[email protected]>
Co-authored-by: Anan Zhuang <[email protected]>
Co-authored-by: Qingyang(Abby) Hu <[email protected]>
Co-authored-by: Zashary Maskus-Lavin <[email protected]>
Co-authored-by: Cameron Reed <[email protected]>
Co-authored-by: Cameron Reed <[email protected]>
(cherry picked from commit 0557ab1)
Signed-off-by: Ashwin P Chandran <[email protected]>
@codecov
Copy link

codecov bot commented Oct 3, 2023

Codecov Report

Merging #5177 (c7b2978) into 2.x (5caba16) will increase coverage by 0.27%.
Report is 2 commits behind head on 2.x.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##              2.x    #5177      +/-   ##
==========================================
+ Coverage   66.53%   66.80%   +0.27%     
==========================================
  Files        3403     3278     -125     
  Lines       65084    63042    -2042     
  Branches    10402    10028     -374     
==========================================
- Hits        43303    42116    -1187     
+ Misses      19211    18467     -744     
+ Partials     2570     2459     -111     
Flag Coverage Δ
Linux_1 35.32% <0.00%> (+0.49%) ⬆️
Linux_2 55.26% <ø> (-0.08%) ⬇️
Linux_3 43.76% <ø> (-0.85%) ⬇️
Linux_4 35.55% <ø> (?)
Windows_1 35.33% <0.00%> (+0.49%) ⬆️
Windows_2 55.23% <ø> (-0.08%) ⬇️
Windows_3 43.76% <ø> (-0.85%) ⬇️
Windows_4 35.55% <ø> (+0.53%) ⬆️

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

Files Coverage Δ
packages/osd-ui-shared-deps/index.js 100.00% <ø> (ø)
src/plugins/dashboard/public/plugin.tsx 0.00% <ø> (ø)
...ugins/data/common/search/aggs/utils/prop_filter.ts 100.00% <ø> (ø)
...s/data/common/search/aggs/utils/to_angular_json.ts 0.00% <ø> (ø)
.../data_explorer/public/components/sidebar/index.tsx 3.12% <ø> (ø)
src/plugins/discover/common/index.ts 100.00% <ø> (ø)
...n/components/doc_viewer_links/doc_viewer_links.tsx 100.00% <ø> (ø)
.../application/components/sidebar/discover_field.tsx 84.84% <ø> (ø)
...rc/plugins/discover/public/embeddable/constants.ts 100.00% <ø> (ø)
src/plugins/discover/public/index.ts 0.00% <ø> (ø)
... and 3 more

... and 23 files with indirect coverage changes

@joshuarrrr
Copy link
Member

looks like a snapshot needs updating

Signed-off-by: Ashwin P Chandran <[email protected]>
@@ -16,7 +16,6 @@
"@osd/i18n": "1.0.0",
"@osd/monaco": "1.0.0",
"abortcontroller-polyfill": "^1.4.0",
"angular": "^1.8.2",
Copy link
Collaborator

Choose a reason for hiding this comment

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

🕺🏼

@AMoo-Miki AMoo-Miki merged commit ab84ea1 into opensearch-project:2.x Oct 4, 2023
53 of 54 checks passed
AMoo-Miki added a commit to AMoo-Miki/OpenSearch-Dashboards that referenced this pull request Oct 4, 2023
@AMoo-Miki AMoo-Miki mentioned this pull request Oct 4, 2023
8 tasks
AMoo-Miki added a commit that referenced this pull request Oct 4, 2023
AMoo-Miki added a commit that referenced this pull request Oct 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autocut Skip the changelog verification check on backports de-angular de-angularize work v2.11.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants