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][CCI] Fix EUI/OUI type errors (#3798) #4861

Merged

Conversation

ashwin-pc
Copy link
Member

cherrypick 0e25f2e from #3798

@codecov
Copy link

codecov bot commented Aug 30, 2023

Codecov Report

Merging #4861 (3bda298) into 2.x (15350bd) will decrease coverage by 0.12%.
Report is 1 commits behind head on 2.x.
The diff coverage is n/a.

❗ Current head 3bda298 differs from pull request most recent head 951de8d. Consider uploading reports for the commit 951de8d to get more accurate results

@@            Coverage Diff             @@
##              2.x    #4861      +/-   ##
==========================================
- Coverage   66.44%   66.32%   -0.12%     
==========================================
  Files        3398     3321      -77     
  Lines       64871    63897     -974     
  Branches    10358    10112     -246     
==========================================
- Hits        43102    42382     -720     
+ Misses      19211    19028     -183     
+ Partials     2558     2487      -71     
Flag Coverage Δ
Linux_1 34.86% <ø> (-0.05%) ⬇️
Linux_2 55.15% <ø> (-0.04%) ⬇️
Linux_3 43.50% <ø> (-0.70%) ⬇️
Linux_4 35.12% <ø> (+0.03%) ⬆️
Windows_1 34.87% <ø> (-0.05%) ⬇️
Windows_2 55.11% <ø> (-0.04%) ⬇️
Windows_3 43.51% <ø> (-0.70%) ⬇️
Windows_4 35.12% <ø> (+0.03%) ⬆️

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

Files Changed Coverage Δ
...public/management_app/components/search/search.tsx 100.00% <ø> (ø)

... and 250 files with indirect coverage changes

@ashwin-pc
Copy link
Member Author

Cypress tests will pass once the Data explorer changes are emerged to 2.x

AMoo-Miki
AMoo-Miki previously approved these changes Aug 31, 2023
@joshuarrrr
Copy link
Member

@joshuarrrr
Copy link
Member

@ashwin-pc This has conflicts now...

manasvinibs
manasvinibs previously approved these changes Sep 1, 2023
@kavilla
Copy link
Member

kavilla commented Sep 1, 2023

@ashwin-pc, lemme know if you need help handling some of these conflicts.

* Update find_test_subject imports for tests

Signed-off-by: Alexei Karikov <[email protected]>

* Update to available imports for findTestSubject

Signed-off-by: Alexei Karikov <[email protected]>

* Fix available import for Query and custom icon

Signed-off-by: Alexei Karikov <[email protected]>

* Add changelog entry

Signed-off-by: Alexei Karikov <[email protected]>

* Add ts-ignore

Signed-off-by: Alexei Karikov <[email protected]>

---------

Signed-off-by: Alexei Karikov <[email protected]>
Co-authored-by: Qingyang(Abby) Hu <[email protected]>
(cherry picked from commit 0e25f2e)
Signed-off-by: Ashwin P Chandran <[email protected]>
@ashwin-pc ashwin-pc dismissed stale reviews from manasvinibs and AMoo-Miki via 951de8d September 2, 2023 00:07
@ashwin-pc ashwin-pc force-pushed the backport/backport-3798-to-2.x branch from 3bda298 to 951de8d Compare September 2, 2023 00:07
@AMoo-Miki AMoo-Miki merged commit 5910a95 into opensearch-project:2.x Sep 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants