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

Removes backport and release-notes scripts #1234

Merged

Conversation

tmarkley
Copy link
Contributor

Description

  • Deletes the unnecessary @osd/release-notes package.
  • Backport is unnecessary since we're adding a backport action in [CI] Add backport action for automated backports #1233.
  • Release notes are generated with git-release-notes, and there's no need to maintain a package dedicated to release notes for now. Future work can be done to integrate existing release notes repositories instead of reinventing the wheel.
  • Removes unused @types/graphql dependency.

Issues Resolved

N/A

Check List

  • Commits are signed per the DCO using --signoff

@tmarkley tmarkley requested a review from a team February 10, 2022 23:40
* Also deletes the unnecessary @osd/release-notes package.
* Backport is unnecessary since we're adding a backport action in opensearch-project#1233.
* Release notes are generated with `git-release-notes`, and there's no
need to maintain a package dedicated to release notes for now. Future
work can be done to integrate existing release notes repositories
instead of reinventing the wheel.

Signed-off-by: Tommy Markley <markleyt@amazon.com>
@tmarkley tmarkley force-pushed the remove_backport_release_notes branch from de76a52 to ecf76e4 Compare February 11, 2022 17:31
@kavilla
Copy link
Member

kavilla commented Feb 15, 2022

Do you think we should create a proposal and leave it open for a little? I know within the OpenSearch project we don't utilize existing features but I don't have the data to say forks of OpenSearch Dashboards do not.

I do not believe people are but just to try to get community insight.

@tmarkley
Copy link
Contributor Author

tmarkley commented Feb 15, 2022

Do you think we should create a proposal and leave it open for a little? I know within the OpenSearch project we don't utilize existing features but I don't have the data to say forks of OpenSearch Dashboards do not.

I do not believe people are but just to try to get community insight.

We could, but I don't see why forks would need to use any of this code. Most of it is specific to the Kibana repo anyways, e.g. the tags that they use for release notes. Also, it'd be easy enough to revive this code and add improvements to it in the process.

Copy link
Member

@ananzh ananzh left a comment

Choose a reason for hiding this comment

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

LGTM

@kavilla
Copy link
Member

kavilla commented Feb 17, 2022

Most of it is specific to the Kibana repo anyways, e.g. the tags that they use for release notes. Also, it'd be easy enough to revive this code and add improvements to it in the process.

Makes sense to me.

@@ -272,7 +271,6 @@
"@types/getos": "^3.0.0",
"@types/glob": "^7.1.3",
"@types/globby": "^8.0.0",
"@types/graphql": "^0.13.2",
Copy link
Member

Choose a reason for hiding this comment

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

Looks like purely an x-pack thing.

Copy link
Member

@kavilla kavilla left a comment

Choose a reason for hiding this comment

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

LGTM.

@tmarkley tmarkley merged commit 91fdc2a into opensearch-project:main Feb 17, 2022
@tmarkley tmarkley deleted the remove_backport_release_notes branch February 17, 2022 16:51
kavilla added a commit to kavilla/OpenSearch-Dashboards-1 that referenced this pull request Feb 18, 2022
openseasrch_config.test.ts was failing with yarn test:jest

Introduced:
opensearch-project#1234

Issue resolved:
n/a

Signed-off-by: Kawika Avilla <kavilla414@gmail.com>
kavilla added a commit that referenced this pull request Feb 21, 2022
openseasrch_config.test.ts was failing with yarn test:jest

Introduced:
#1234

Issue resolved:
n/a

Signed-off-by: Kawika Avilla <kavilla414@gmail.com>
@tmarkley tmarkley added v2.0.0 dependencies Pull requests that update a dependency file docs Improvements or additions to documentation labels Mar 1, 2022
ashwin-pc added a commit to ashwin-pc/OpenSearch-Dashboards that referenced this pull request Feb 7, 2024
### Description

updates changelog

### Issues Resolved

<!-- List any issues this PR will resolve. Prefix the issue with the keyword closes, fixes, fix -->
<!-- Example: closes opensearch-project#1234 or fixes <Issue_URL> -->

## Screenshot

<!-- Attach any relevant screenshots. Any change to the UI requires an attached screenshot in the PR Description -->

## Testing the changes

<!--
  Please provide detailed steps for validating your changes. This could involve specific commands to run,
  pages to visit, scenarios to try or any other information that would help reviewers verify
  the functionality of your change
-->

### Check List

- [ ] All tests pass
  - [ ] `yarn test:jest`
  - [ ] `yarn test:jest_integration`
- [ ] New functionality includes testing.
- [ ] New functionality has been documented.
- [ ] Update [CHANGELOG.md](./../CHANGELOG.md)
- [ ] Commits are signed per the DCO using --signoff
ashwin-pc added a commit that referenced this pull request Feb 7, 2024
* Update table revert changelog
### Description

updates changelog

### Issues Resolved

<!-- List any issues this PR will resolve. Prefix the issue with the keyword closes, fixes, fix -->
<!-- Example: closes #1234 or fixes <Issue_URL> -->

## Screenshot

<!-- Attach any relevant screenshots. Any change to the UI requires an attached screenshot in the PR Description -->

## Testing the changes

<!--
  Please provide detailed steps for validating your changes. This could involve specific commands to run,
  pages to visit, scenarios to try or any other information that would help reviewers verify
  the functionality of your change
-->

### Check List

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

* revert dependency change

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* nit

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* update changelog

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

---------

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>
ashwin-pc added a commit that referenced this pull request Feb 7, 2024
…5789)

* basic table working

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* basic styling

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix header column style

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* correctly display optional col header actions

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* move col to the right or left if applicable

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* add cell filter for and filter out buttons

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* add cell action on doc table, half working on toggle columns

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* sorting

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Add toggling between legacy and new table

According to #5739, add toggle using storage.

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* get rid of the console log

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* optimize cell filter logics

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* add more refactoring

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* remove an used line

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* some more table cell optimization

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Initial UI fixes (#5758)
Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Table optimization with lazy loading (#5760)

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* [Discover with Legacy Table] fix copy issue and open feedbacks in a new tab (#5761)

* add a colon (:) after the key
* ensure there's a space between each key-value pair
* allow feedback url to open in a new tab

Signed-off-by: Anan Z <ananzh@amazon.com>

* [Discover] Fix more UI (#5764)

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* [Discover] Fixes table optimization (#5763)

* [Discover] Fixes table optimization

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* improved loader

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

---------

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Change sort icon according to current sort conditions (#5772)

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix breadcrumb navigation (#5773)

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Add pagination to discover embeddable (#5770)

* Add pagination

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix an errir

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* small change

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix errors

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

---------

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* [Discover] Prevent wrapping of time series cells (#5779)

Signed-off-by: Miki <miki@amazon.com>

* Vertically align the text in QueryStringInputUI with other elements on the page (#5780)

Signed-off-by: Miki <miki@amazon.com>

* Use `ouiCodeFont` in Discover and reduce text size (#5783)

Also:
* Fix react `key` errors
* vertically align source cells
* Clamp the height of the cells
* Make sure time-series column never grows

Signed-off-by: Miki <miki@amazon.com>

* [Discover] Display filter buttons at top right of the cell (#5784)

Signed-off-by: Miki <miki@amazon.com>

* [Discover] Vertically align the details toggle button (#5785)

Signed-off-by: Miki <miki@amazon.com>

* [Discover] Fix overflow of the expanded document (#5788)

* Fix vertical alignment of expand details button
Fix colspan of details cells

Signed-off-by: Miki <miki@amazon.com>

* Fix overflow problem of detailed doc

Signed-off-by: Miki <miki@amazon.com>

---------

Signed-off-by: Miki <miki@amazon.com>

* update feedback msg (#5787)

Signed-off-by: Anan Z <ananzh@amazon.com>

* Resolve sort, default sort and short dot (#5771)

Signed-off-by: Anan Z <ananzh@amazon.com>
Signed-off-by: Anan Zhuang <ananzh@amazon.com>

* [Discover] Fixes pagination style (#5778)

* fixes pagination style

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* style-lint fix

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* fix react warning for unnecessary prop

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Show total hit count and pass services in embeddable

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* removed unnecessary div

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

---------

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>
Signed-off-by: Anan Zhuang <ananzh@amazon.com>
Co-authored-by: Anan Zhuang <ananzh@amazon.com>

* add changelog

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix table column error

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Fix pagination edge case

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Routing for surrounding doc link should work without a question mark appending to the end (#5776)

* routing for surrounding doc should work without ?

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* change path to not include appstate

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* delete optional app state for single doc link too

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

---------

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* type fixes (#5795)

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Update toggle to button (#5808)

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Adds support for line count (#5814)

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Add swith to datagrid table and fix ciGroup1 (#5816)

* add swith to datagrid table and fix ciGroup1

Signed-off-by: Anan <ananzh@amazon.com>

* fix ciGroup7

Signed-off-by: Anan Z <ananzh@amazon.com>

* fix ciGroup6

Signed-off-by: Anan Z <ananzh@amazon.com>

---------

Signed-off-by: Anan <ananzh@amazon.com>
Signed-off-by: Anan Z <ananzh@amazon.com>

* add missing snapshot (#5818)

Signed-off-by: Anan Z <ananzh@amazon.com>

* fix pluign ftr tests (#5821)

Signed-off-by: Anan Z <ananzh@amazon.com>

* update snapshot (#5824)

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Fix functional 3 and 4 for discover legacy (#5822)

* fix 3 and 4

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix functional 3 and 4

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* dashboard expect

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

---------

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Solve cigroup 7 flakiness (#5826)

* test hover

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* comment out other ci to speed up

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* uncomment

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

---------

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* [Discover] Simplify feedback modal (#5837)

* Simplify feedback modal

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* renames datagrid  settng to newDiscover setting

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* removes sub module from branch

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

---------

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Persist line height (#5836)

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Update table revert changelog (#5835)

* Update table revert changelog
### Description

updates changelog

### Issues Resolved

<!-- List any issues this PR will resolve. Prefix the issue with the keyword closes, fixes, fix -->
<!-- Example: closes #1234 or fixes <Issue_URL> -->

## Screenshot

<!-- Attach any relevant screenshots. Any change to the UI requires an attached screenshot in the PR Description -->

## Testing the changes

<!--
  Please provide detailed steps for validating your changes. This could involve specific commands to run,
  pages to visit, scenarios to try or any other information that would help reviewers verify
  the functionality of your change
-->

### Check List

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

* revert dependency change

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* nit

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* update changelog

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

---------

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* fix table height persist on context page

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

---------

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>
Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>
Signed-off-by: Anan Z <ananzh@amazon.com>
Signed-off-by: Miki <miki@amazon.com>
Signed-off-by: Anan Zhuang <ananzh@amazon.com>
Signed-off-by: Anan <ananzh@amazon.com>
Co-authored-by: Ashwin P Chandran <ashwinpc@amazon.com>
Co-authored-by: Anan Zhuang <ananzh@amazon.com>
Co-authored-by: Miki <miki@amazon.com>
Co-authored-by: Ganesh0107 <ganesh.gopal@freshworks.com>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Feb 7, 2024
…5789)

* basic table working

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* basic styling

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix header column style

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* correctly display optional col header actions

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* move col to the right or left if applicable

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* add cell filter for and filter out buttons

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* add cell action on doc table, half working on toggle columns

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* sorting

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Add toggling between legacy and new table

According to #5739, add toggle using storage.

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* get rid of the console log

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* optimize cell filter logics

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* add more refactoring

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* remove an used line

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* some more table cell optimization

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Initial UI fixes (#5758)
Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Table optimization with lazy loading (#5760)

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* [Discover with Legacy Table] fix copy issue and open feedbacks in a new tab (#5761)

* add a colon (:) after the key
* ensure there's a space between each key-value pair
* allow feedback url to open in a new tab

Signed-off-by: Anan Z <ananzh@amazon.com>

* [Discover] Fix more UI (#5764)

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* [Discover] Fixes table optimization (#5763)

* [Discover] Fixes table optimization

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* improved loader

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

---------

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Change sort icon according to current sort conditions (#5772)

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix breadcrumb navigation (#5773)

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Add pagination to discover embeddable (#5770)

* Add pagination

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix an errir

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* small change

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix errors

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

---------

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* [Discover] Prevent wrapping of time series cells (#5779)

Signed-off-by: Miki <miki@amazon.com>

* Vertically align the text in QueryStringInputUI with other elements on the page (#5780)

Signed-off-by: Miki <miki@amazon.com>

* Use `ouiCodeFont` in Discover and reduce text size (#5783)

Also:
* Fix react `key` errors
* vertically align source cells
* Clamp the height of the cells
* Make sure time-series column never grows

Signed-off-by: Miki <miki@amazon.com>

* [Discover] Display filter buttons at top right of the cell (#5784)

Signed-off-by: Miki <miki@amazon.com>

* [Discover] Vertically align the details toggle button (#5785)

Signed-off-by: Miki <miki@amazon.com>

* [Discover] Fix overflow of the expanded document (#5788)

* Fix vertical alignment of expand details button
Fix colspan of details cells

Signed-off-by: Miki <miki@amazon.com>

* Fix overflow problem of detailed doc

Signed-off-by: Miki <miki@amazon.com>

---------

Signed-off-by: Miki <miki@amazon.com>

* update feedback msg (#5787)

Signed-off-by: Anan Z <ananzh@amazon.com>

* Resolve sort, default sort and short dot (#5771)

Signed-off-by: Anan Z <ananzh@amazon.com>
Signed-off-by: Anan Zhuang <ananzh@amazon.com>

* [Discover] Fixes pagination style (#5778)

* fixes pagination style

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* style-lint fix

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* fix react warning for unnecessary prop

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Show total hit count and pass services in embeddable

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* removed unnecessary div

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

---------

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>
Signed-off-by: Anan Zhuang <ananzh@amazon.com>
Co-authored-by: Anan Zhuang <ananzh@amazon.com>

* add changelog

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix table column error

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Fix pagination edge case

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Routing for surrounding doc link should work without a question mark appending to the end (#5776)

* routing for surrounding doc should work without ?

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* change path to not include appstate

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* delete optional app state for single doc link too

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

---------

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* type fixes (#5795)

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Update toggle to button (#5808)

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Adds support for line count (#5814)

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Add swith to datagrid table and fix ciGroup1 (#5816)

* add swith to datagrid table and fix ciGroup1

Signed-off-by: Anan <ananzh@amazon.com>

* fix ciGroup7

Signed-off-by: Anan Z <ananzh@amazon.com>

* fix ciGroup6

Signed-off-by: Anan Z <ananzh@amazon.com>

---------

Signed-off-by: Anan <ananzh@amazon.com>
Signed-off-by: Anan Z <ananzh@amazon.com>

* add missing snapshot (#5818)

Signed-off-by: Anan Z <ananzh@amazon.com>

* fix pluign ftr tests (#5821)

Signed-off-by: Anan Z <ananzh@amazon.com>

* update snapshot (#5824)

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Fix functional 3 and 4 for discover legacy (#5822)

* fix 3 and 4

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix functional 3 and 4

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* dashboard expect

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

---------

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Solve cigroup 7 flakiness (#5826)

* test hover

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* comment out other ci to speed up

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* uncomment

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

---------

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* [Discover] Simplify feedback modal (#5837)

* Simplify feedback modal

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* renames datagrid  settng to newDiscover setting

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* removes sub module from branch

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

---------

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Persist line height (#5836)

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Update table revert changelog (#5835)

* Update table revert changelog
### Description

updates changelog

### Issues Resolved

<!-- List any issues this PR will resolve. Prefix the issue with the keyword closes, fixes, fix -->
<!-- Example: closes #1234 or fixes <Issue_URL> -->

## Screenshot

<!-- Attach any relevant screenshots. Any change to the UI requires an attached screenshot in the PR Description -->

## Testing the changes

<!--
  Please provide detailed steps for validating your changes. This could involve specific commands to run,
  pages to visit, scenarios to try or any other information that would help reviewers verify
  the functionality of your change
-->

### Check List

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

* revert dependency change

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* nit

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* update changelog

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

---------

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* fix table height persist on context page

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

---------

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>
Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>
Signed-off-by: Anan Z <ananzh@amazon.com>
Signed-off-by: Miki <miki@amazon.com>
Signed-off-by: Anan Zhuang <ananzh@amazon.com>
Signed-off-by: Anan <ananzh@amazon.com>
Co-authored-by: Ashwin P Chandran <ashwinpc@amazon.com>
Co-authored-by: Anan Zhuang <ananzh@amazon.com>
Co-authored-by: Miki <miki@amazon.com>
Co-authored-by: Ganesh0107 <ganesh.gopal@freshworks.com>
(cherry picked from commit e13dbff)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
ashwin-pc pushed a commit to ashwin-pc/OpenSearch-Dashboards that referenced this pull request Feb 7, 2024
…pensearch-project#5789)

* basic table working

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* basic styling

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix header column style

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* correctly display optional col header actions

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* move col to the right or left if applicable

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* add cell filter for and filter out buttons

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* add cell action on doc table, half working on toggle columns

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* sorting

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Add toggling between legacy and new table

According to opensearch-project#5739, add toggle using storage.

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* get rid of the console log

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* optimize cell filter logics

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* add more refactoring

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* remove an used line

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* some more table cell optimization

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Initial UI fixes (opensearch-project#5758)
Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Table optimization with lazy loading (opensearch-project#5760)

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* [Discover with Legacy Table] fix copy issue and open feedbacks in a new tab (opensearch-project#5761)

* add a colon (:) after the key
* ensure there's a space between each key-value pair
* allow feedback url to open in a new tab

Signed-off-by: Anan Z <ananzh@amazon.com>

* [Discover] Fix more UI (opensearch-project#5764)

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* [Discover] Fixes table optimization (opensearch-project#5763)

* [Discover] Fixes table optimization

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* improved loader

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

---------

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Change sort icon according to current sort conditions (opensearch-project#5772)

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix breadcrumb navigation (opensearch-project#5773)

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Add pagination to discover embeddable (opensearch-project#5770)

* Add pagination

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix an errir

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* small change

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix errors

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

---------

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* [Discover] Prevent wrapping of time series cells (opensearch-project#5779)

Signed-off-by: Miki <miki@amazon.com>

* Vertically align the text in QueryStringInputUI with other elements on the page (opensearch-project#5780)

Signed-off-by: Miki <miki@amazon.com>

* Use `ouiCodeFont` in Discover and reduce text size (opensearch-project#5783)

Also:
* Fix react `key` errors
* vertically align source cells
* Clamp the height of the cells
* Make sure time-series column never grows

Signed-off-by: Miki <miki@amazon.com>

* [Discover] Display filter buttons at top right of the cell (opensearch-project#5784)

Signed-off-by: Miki <miki@amazon.com>

* [Discover] Vertically align the details toggle button (opensearch-project#5785)

Signed-off-by: Miki <miki@amazon.com>

* [Discover] Fix overflow of the expanded document (opensearch-project#5788)

* Fix vertical alignment of expand details button
Fix colspan of details cells

Signed-off-by: Miki <miki@amazon.com>

* Fix overflow problem of detailed doc

Signed-off-by: Miki <miki@amazon.com>

---------

Signed-off-by: Miki <miki@amazon.com>

* update feedback msg (opensearch-project#5787)

Signed-off-by: Anan Z <ananzh@amazon.com>

* Resolve sort, default sort and short dot (opensearch-project#5771)

Signed-off-by: Anan Z <ananzh@amazon.com>
Signed-off-by: Anan Zhuang <ananzh@amazon.com>

* [Discover] Fixes pagination style (opensearch-project#5778)

* fixes pagination style

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* style-lint fix

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* fix react warning for unnecessary prop

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Show total hit count and pass services in embeddable

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* removed unnecessary div

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

---------

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>
Signed-off-by: Anan Zhuang <ananzh@amazon.com>
Co-authored-by: Anan Zhuang <ananzh@amazon.com>

* add changelog

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix table column error

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Fix pagination edge case

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Routing for surrounding doc link should work without a question mark appending to the end (opensearch-project#5776)

* routing for surrounding doc should work without ?

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* change path to not include appstate

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* delete optional app state for single doc link too

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

---------

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* type fixes (opensearch-project#5795)

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Update toggle to button (opensearch-project#5808)

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Adds support for line count (opensearch-project#5814)

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Add swith to datagrid table and fix ciGroup1 (opensearch-project#5816)

* add swith to datagrid table and fix ciGroup1

Signed-off-by: Anan <ananzh@amazon.com>

* fix ciGroup7

Signed-off-by: Anan Z <ananzh@amazon.com>

* fix ciGroup6

Signed-off-by: Anan Z <ananzh@amazon.com>

---------

Signed-off-by: Anan <ananzh@amazon.com>
Signed-off-by: Anan Z <ananzh@amazon.com>

* add missing snapshot (opensearch-project#5818)

Signed-off-by: Anan Z <ananzh@amazon.com>

* fix pluign ftr tests (opensearch-project#5821)

Signed-off-by: Anan Z <ananzh@amazon.com>

* update snapshot (opensearch-project#5824)

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Fix functional 3 and 4 for discover legacy (opensearch-project#5822)

* fix 3 and 4

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix functional 3 and 4

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* dashboard expect

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

---------

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Solve cigroup 7 flakiness (opensearch-project#5826)

* test hover

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* comment out other ci to speed up

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* uncomment

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

---------

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* [Discover] Simplify feedback modal (opensearch-project#5837)

* Simplify feedback modal

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* renames datagrid  settng to newDiscover setting

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* removes sub module from branch

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

---------

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Persist line height (opensearch-project#5836)

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Update table revert changelog (opensearch-project#5835)

* Update table revert changelog
### Description

updates changelog

### Issues Resolved

<!-- List any issues this PR will resolve. Prefix the issue with the keyword closes, fixes, fix -->
<!-- Example: closes opensearch-project#1234 or fixes <Issue_URL> -->

## Screenshot

<!-- Attach any relevant screenshots. Any change to the UI requires an attached screenshot in the PR Description -->

## Testing the changes

<!--
  Please provide detailed steps for validating your changes. This could involve specific commands to run,
  pages to visit, scenarios to try or any other information that would help reviewers verify
  the functionality of your change
-->

### Check List

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

* revert dependency change

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* nit

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* update changelog

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

---------

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* fix table height persist on context page

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

---------

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>
Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>
Signed-off-by: Anan Z <ananzh@amazon.com>
Signed-off-by: Miki <miki@amazon.com>
Signed-off-by: Anan Zhuang <ananzh@amazon.com>
Signed-off-by: Anan <ananzh@amazon.com>
Co-authored-by: Ashwin P Chandran <ashwinpc@amazon.com>
Co-authored-by: Anan Zhuang <ananzh@amazon.com>
Co-authored-by: Miki <miki@amazon.com>
Co-authored-by: Ganesh0107 <ganesh.gopal@freshworks.com>
(cherry picked from commit e13dbff)
ashwin-pc added a commit that referenced this pull request Feb 8, 2024
…5789) (#5840)

* basic table working

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* basic styling

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix header column style

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* correctly display optional col header actions

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* move col to the right or left if applicable

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* add cell filter for and filter out buttons

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* add cell action on doc table, half working on toggle columns

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* sorting

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Add toggling between legacy and new table

According to #5739, add toggle using storage.

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* get rid of the console log

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* optimize cell filter logics

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* add more refactoring

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* remove an used line

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* some more table cell optimization

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Initial UI fixes (#5758)
Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Table optimization with lazy loading (#5760)

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* [Discover with Legacy Table] fix copy issue and open feedbacks in a new tab (#5761)

* add a colon (:) after the key
* ensure there's a space between each key-value pair
* allow feedback url to open in a new tab

Signed-off-by: Anan Z <ananzh@amazon.com>

* [Discover] Fix more UI (#5764)

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* [Discover] Fixes table optimization (#5763)

* [Discover] Fixes table optimization

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* improved loader

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

---------

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Change sort icon according to current sort conditions (#5772)

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix breadcrumb navigation (#5773)

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Add pagination to discover embeddable (#5770)

* Add pagination

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix an errir

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* small change

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix errors

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

---------

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* [Discover] Prevent wrapping of time series cells (#5779)

Signed-off-by: Miki <miki@amazon.com>

* Vertically align the text in QueryStringInputUI with other elements on the page (#5780)

Signed-off-by: Miki <miki@amazon.com>

* Use `ouiCodeFont` in Discover and reduce text size (#5783)

Also:
* Fix react `key` errors
* vertically align source cells
* Clamp the height of the cells
* Make sure time-series column never grows

Signed-off-by: Miki <miki@amazon.com>

* [Discover] Display filter buttons at top right of the cell (#5784)

Signed-off-by: Miki <miki@amazon.com>

* [Discover] Vertically align the details toggle button (#5785)

Signed-off-by: Miki <miki@amazon.com>

* [Discover] Fix overflow of the expanded document (#5788)

* Fix vertical alignment of expand details button
Fix colspan of details cells

Signed-off-by: Miki <miki@amazon.com>

* Fix overflow problem of detailed doc

Signed-off-by: Miki <miki@amazon.com>

---------

Signed-off-by: Miki <miki@amazon.com>

* update feedback msg (#5787)

Signed-off-by: Anan Z <ananzh@amazon.com>

* Resolve sort, default sort and short dot (#5771)

Signed-off-by: Anan Z <ananzh@amazon.com>
Signed-off-by: Anan Zhuang <ananzh@amazon.com>

* [Discover] Fixes pagination style (#5778)

* fixes pagination style

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* style-lint fix

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* fix react warning for unnecessary prop

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Show total hit count and pass services in embeddable

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* removed unnecessary div

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

---------

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>
Signed-off-by: Anan Zhuang <ananzh@amazon.com>
Co-authored-by: Anan Zhuang <ananzh@amazon.com>

* add changelog

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix table column error

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Fix pagination edge case

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Routing for surrounding doc link should work without a question mark appending to the end (#5776)

* routing for surrounding doc should work without ?

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* change path to not include appstate

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* delete optional app state for single doc link too

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

---------

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* type fixes (#5795)

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Update toggle to button (#5808)

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Adds support for line count (#5814)

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Add swith to datagrid table and fix ciGroup1 (#5816)

* add swith to datagrid table and fix ciGroup1

Signed-off-by: Anan <ananzh@amazon.com>

* fix ciGroup7

Signed-off-by: Anan Z <ananzh@amazon.com>

* fix ciGroup6

Signed-off-by: Anan Z <ananzh@amazon.com>

---------

Signed-off-by: Anan <ananzh@amazon.com>
Signed-off-by: Anan Z <ananzh@amazon.com>

* add missing snapshot (#5818)

Signed-off-by: Anan Z <ananzh@amazon.com>

* fix pluign ftr tests (#5821)

Signed-off-by: Anan Z <ananzh@amazon.com>

* update snapshot (#5824)

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Fix functional 3 and 4 for discover legacy (#5822)

* fix 3 and 4

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix functional 3 and 4

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* dashboard expect

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

---------

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Solve cigroup 7 flakiness (#5826)

* test hover

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* comment out other ci to speed up

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* uncomment

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

---------

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* [Discover] Simplify feedback modal (#5837)

* Simplify feedback modal

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* renames datagrid  settng to newDiscover setting

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* removes sub module from branch

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

---------

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Persist line height (#5836)

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Update table revert changelog (#5835)

* Update table revert changelog
### Description

updates changelog

### Issues Resolved

<!-- List any issues this PR will resolve. Prefix the issue with the keyword closes, fixes, fix -->
<!-- Example: closes #1234 or fixes <Issue_URL> -->

## Screenshot

<!-- Attach any relevant screenshots. Any change to the UI requires an attached screenshot in the PR Description -->

## Testing the changes

<!--
  Please provide detailed steps for validating your changes. This could involve specific commands to run,
  pages to visit, scenarios to try or any other information that would help reviewers verify
  the functionality of your change
-->

### Check List

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

* revert dependency change

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* nit

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* update changelog

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

---------

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* fix table height persist on context page

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

---------

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>
Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>
Signed-off-by: Anan Z <ananzh@amazon.com>
Signed-off-by: Miki <miki@amazon.com>
Signed-off-by: Anan Zhuang <ananzh@amazon.com>
Signed-off-by: Anan <ananzh@amazon.com>
Co-authored-by: Ashwin P Chandran <ashwinpc@amazon.com>
Co-authored-by: Anan Zhuang <ananzh@amazon.com>
Co-authored-by: Miki <miki@amazon.com>
Co-authored-by: Ganesh0107 <ganesh.gopal@freshworks.com>
(cherry picked from commit e13dbff)

Co-authored-by: Qingyang(Abby) Hu <abigailhu2000@gmail.com>
kavilla pushed a commit that referenced this pull request Feb 8, 2024
…5789) (#5839)

* basic table working



* basic styling



* fix header column style



* correctly display optional col header actions



* move col to the right or left if applicable



* add cell filter for and filter out buttons



* add cell action on doc table, half working on toggle columns



* sorting



* Add toggling between legacy and new table

According to #5739, add toggle using storage.



* get rid of the console log



* optimize cell filter logics



* add more refactoring



* remove an used line



* some more table cell optimization



* Initial UI fixes (#5758)


* Table optimization with lazy loading (#5760)



* [Discover with Legacy Table] fix copy issue and open feedbacks in a new tab (#5761)

* add a colon (:) after the key
* ensure there's a space between each key-value pair
* allow feedback url to open in a new tab



* [Discover] Fix more UI (#5764)



* [Discover] Fixes table optimization (#5763)

* [Discover] Fixes table optimization



* improved loader



---------



* Change sort icon according to current sort conditions (#5772)



* fix breadcrumb navigation (#5773)



* Add pagination to discover embeddable (#5770)

* Add pagination



* fix an errir



* small change



* fix errors



---------



* [Discover] Prevent wrapping of time series cells (#5779)



* Vertically align the text in QueryStringInputUI with other elements on the page (#5780)



* Use `ouiCodeFont` in Discover and reduce text size (#5783)

Also:
* Fix react `key` errors
* vertically align source cells
* Clamp the height of the cells
* Make sure time-series column never grows



* [Discover] Display filter buttons at top right of the cell (#5784)



* [Discover] Vertically align the details toggle button (#5785)



* [Discover] Fix overflow of the expanded document (#5788)

* Fix vertical alignment of expand details button
Fix colspan of details cells



* Fix overflow problem of detailed doc



---------



* update feedback msg (#5787)



* Resolve sort, default sort and short dot (#5771)




* [Discover] Fixes pagination style (#5778)

* fixes pagination style



* style-lint fix



* fix react warning for unnecessary prop



* Show total hit count and pass services in embeddable



* removed unnecessary div



---------





* add changelog



* fix table column error



* Fix pagination edge case



* Routing for surrounding doc link should work without a question mark appending to the end (#5776)

* routing for surrounding doc should work without ?



* change path to not include appstate



* delete optional app state for single doc link too



---------



* type fixes (#5795)



* Update toggle to button (#5808)



* Adds support for line count (#5814)



* Add swith to datagrid table and fix ciGroup1 (#5816)

* add swith to datagrid table and fix ciGroup1



* fix ciGroup7



* fix ciGroup6



---------




* add missing snapshot (#5818)



* fix pluign ftr tests (#5821)



* update snapshot (#5824)



* Fix functional 3 and 4 for discover legacy (#5822)

* fix 3 and 4



* fix functional 3 and 4



* dashboard expect



---------



* Solve cigroup 7 flakiness (#5826)

* test hover



* comment out other ci to speed up



* uncomment



---------



* [Discover] Simplify feedback modal (#5837)

* Simplify feedback modal



* renames datagrid  settng to newDiscover setting



* removes sub module from branch



---------



* Persist line height (#5836)



* Update table revert changelog (#5835)

* Update table revert changelog
### Description

updates changelog

### Issues Resolved

<!-- List any issues this PR will resolve. Prefix the issue with the keyword closes, fixes, fix -->
<!-- Example: closes #1234 or fixes <Issue_URL> -->

## Screenshot

<!-- Attach any relevant screenshots. Any change to the UI requires an attached screenshot in the PR Description -->

## Testing the changes

<!--
  Please provide detailed steps for validating your changes. This could involve specific commands to run,
  pages to visit, scenarios to try or any other information that would help reviewers verify
  the functionality of your change
-->

### Check List

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

* revert dependency change



* nit



* update changelog



---------



* fix table height persist on context page



---------











(cherry picked from commit e13dbff)

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>
Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>
Signed-off-by: Anan Z <ananzh@amazon.com>
Signed-off-by: Miki <miki@amazon.com>
Signed-off-by: Anan Zhuang <ananzh@amazon.com>
Signed-off-by: Anan <ananzh@amazon.com>
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: Ashwin P Chandran <ashwinpc@amazon.com>
Co-authored-by: Anan Zhuang <ananzh@amazon.com>
Co-authored-by: Miki <miki@amazon.com>
Co-authored-by: Ganesh0107 <ganesh.gopal@freshworks.com>
ananzh added a commit to ananzh/OpenSearch-Dashboards that referenced this pull request Feb 21, 2024
…pensearch-project#5789)

* basic table working

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* basic styling

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix header column style

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* correctly display optional col header actions

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* move col to the right or left if applicable

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* add cell filter for and filter out buttons

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* add cell action on doc table, half working on toggle columns

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* sorting

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Add toggling between legacy and new table

According to opensearch-project#5739, add toggle using storage.

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* get rid of the console log

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* optimize cell filter logics

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* add more refactoring

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* remove an used line

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* some more table cell optimization

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Initial UI fixes (opensearch-project#5758)
Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Table optimization with lazy loading (opensearch-project#5760)

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* [Discover with Legacy Table] fix copy issue and open feedbacks in a new tab (opensearch-project#5761)

* add a colon (:) after the key
* ensure there's a space between each key-value pair
* allow feedback url to open in a new tab

Signed-off-by: Anan Z <ananzh@amazon.com>

* [Discover] Fix more UI (opensearch-project#5764)

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* [Discover] Fixes table optimization (opensearch-project#5763)

* [Discover] Fixes table optimization

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* improved loader

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

---------

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Change sort icon according to current sort conditions (opensearch-project#5772)

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix breadcrumb navigation (opensearch-project#5773)

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Add pagination to discover embeddable (opensearch-project#5770)

* Add pagination

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix an errir

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* small change

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix errors

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

---------

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* [Discover] Prevent wrapping of time series cells (opensearch-project#5779)

Signed-off-by: Miki <miki@amazon.com>

* Vertically align the text in QueryStringInputUI with other elements on the page (opensearch-project#5780)

Signed-off-by: Miki <miki@amazon.com>

* Use `ouiCodeFont` in Discover and reduce text size (opensearch-project#5783)

Also:
* Fix react `key` errors
* vertically align source cells
* Clamp the height of the cells
* Make sure time-series column never grows

Signed-off-by: Miki <miki@amazon.com>

* [Discover] Display filter buttons at top right of the cell (opensearch-project#5784)

Signed-off-by: Miki <miki@amazon.com>

* [Discover] Vertically align the details toggle button (opensearch-project#5785)

Signed-off-by: Miki <miki@amazon.com>

* [Discover] Fix overflow of the expanded document (opensearch-project#5788)

* Fix vertical alignment of expand details button
Fix colspan of details cells

Signed-off-by: Miki <miki@amazon.com>

* Fix overflow problem of detailed doc

Signed-off-by: Miki <miki@amazon.com>

---------

Signed-off-by: Miki <miki@amazon.com>

* update feedback msg (opensearch-project#5787)

Signed-off-by: Anan Z <ananzh@amazon.com>

* Resolve sort, default sort and short dot (opensearch-project#5771)

Signed-off-by: Anan Z <ananzh@amazon.com>
Signed-off-by: Anan Zhuang <ananzh@amazon.com>

* [Discover] Fixes pagination style (opensearch-project#5778)

* fixes pagination style

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* style-lint fix

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* fix react warning for unnecessary prop

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Show total hit count and pass services in embeddable

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* removed unnecessary div

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

---------

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>
Signed-off-by: Anan Zhuang <ananzh@amazon.com>
Co-authored-by: Anan Zhuang <ananzh@amazon.com>

* add changelog

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix table column error

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Fix pagination edge case

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Routing for surrounding doc link should work without a question mark appending to the end (opensearch-project#5776)

* routing for surrounding doc should work without ?

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* change path to not include appstate

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* delete optional app state for single doc link too

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

---------

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* type fixes (opensearch-project#5795)

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Update toggle to button (opensearch-project#5808)

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Adds support for line count (opensearch-project#5814)

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Add swith to datagrid table and fix ciGroup1 (opensearch-project#5816)

* add swith to datagrid table and fix ciGroup1

Signed-off-by: Anan <ananzh@amazon.com>

* fix ciGroup7

Signed-off-by: Anan Z <ananzh@amazon.com>

* fix ciGroup6

Signed-off-by: Anan Z <ananzh@amazon.com>

---------

Signed-off-by: Anan <ananzh@amazon.com>
Signed-off-by: Anan Z <ananzh@amazon.com>

* add missing snapshot (opensearch-project#5818)

Signed-off-by: Anan Z <ananzh@amazon.com>

* fix pluign ftr tests (opensearch-project#5821)

Signed-off-by: Anan Z <ananzh@amazon.com>

* update snapshot (opensearch-project#5824)

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Fix functional 3 and 4 for discover legacy (opensearch-project#5822)

* fix 3 and 4

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix functional 3 and 4

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* dashboard expect

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

---------

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Solve cigroup 7 flakiness (opensearch-project#5826)

* test hover

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* comment out other ci to speed up

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* uncomment

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

---------

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* [Discover] Simplify feedback modal (opensearch-project#5837)

* Simplify feedback modal

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* renames datagrid  settng to newDiscover setting

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* removes sub module from branch

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

---------

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Persist line height (opensearch-project#5836)

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Update table revert changelog (opensearch-project#5835)

* Update table revert changelog

updates changelog

<!-- List any issues this PR will resolve. Prefix the issue with the keyword closes, fixes, fix -->
<!-- Example: closes opensearch-project#1234 or fixes <Issue_URL> -->

<!-- Attach any relevant screenshots. Any change to the UI requires an attached screenshot in the PR Description -->

<!--
  Please provide detailed steps for validating your changes. This could involve specific commands to run,
  pages to visit, scenarios to try or any other information that would help reviewers verify
  the functionality of your change
-->

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

* revert dependency change

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* nit

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* update changelog

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

---------

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* fix table height persist on context page

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

---------

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>
Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>
Signed-off-by: Anan Z <ananzh@amazon.com>
Signed-off-by: Miki <miki@amazon.com>
Signed-off-by: Anan Zhuang <ananzh@amazon.com>
Signed-off-by: Anan <ananzh@amazon.com>
Co-authored-by: Ashwin P Chandran <ashwinpc@amazon.com>
Co-authored-by: Anan Zhuang <ananzh@amazon.com>
Co-authored-by: Miki <miki@amazon.com>
Co-authored-by: Ganesh0107 <ganesh.gopal@freshworks.com>
ananzh added a commit to ananzh/OpenSearch-Dashboards that referenced this pull request Feb 21, 2024
…pensearch-project#5789)

* basic table working

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* basic styling

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix header column style

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* correctly display optional col header actions

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* move col to the right or left if applicable

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* add cell filter for and filter out buttons

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* add cell action on doc table, half working on toggle columns

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* sorting

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Add toggling between legacy and new table

According to opensearch-project#5739, add toggle using storage.

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* get rid of the console log

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* optimize cell filter logics

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* add more refactoring

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* remove an used line

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* some more table cell optimization

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Initial UI fixes (opensearch-project#5758)
Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Table optimization with lazy loading (opensearch-project#5760)

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* [Discover with Legacy Table] fix copy issue and open feedbacks in a new tab (opensearch-project#5761)

* add a colon (:) after the key
* ensure there's a space between each key-value pair
* allow feedback url to open in a new tab

Signed-off-by: Anan Z <ananzh@amazon.com>

* [Discover] Fix more UI (opensearch-project#5764)

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* [Discover] Fixes table optimization (opensearch-project#5763)

* [Discover] Fixes table optimization

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* improved loader

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

---------

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Change sort icon according to current sort conditions (opensearch-project#5772)

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix breadcrumb navigation (opensearch-project#5773)

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Add pagination to discover embeddable (opensearch-project#5770)

* Add pagination

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix an errir

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* small change

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix errors

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

---------

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* [Discover] Prevent wrapping of time series cells (opensearch-project#5779)

Signed-off-by: Miki <miki@amazon.com>

* Vertically align the text in QueryStringInputUI with other elements on the page (opensearch-project#5780)

Signed-off-by: Miki <miki@amazon.com>

* Use `ouiCodeFont` in Discover and reduce text size (opensearch-project#5783)

Also:
* Fix react `key` errors
* vertically align source cells
* Clamp the height of the cells
* Make sure time-series column never grows

Signed-off-by: Miki <miki@amazon.com>

* [Discover] Display filter buttons at top right of the cell (opensearch-project#5784)

Signed-off-by: Miki <miki@amazon.com>

* [Discover] Vertically align the details toggle button (opensearch-project#5785)

Signed-off-by: Miki <miki@amazon.com>

* [Discover] Fix overflow of the expanded document (opensearch-project#5788)

* Fix vertical alignment of expand details button
Fix colspan of details cells

Signed-off-by: Miki <miki@amazon.com>

* Fix overflow problem of detailed doc

Signed-off-by: Miki <miki@amazon.com>

---------

Signed-off-by: Miki <miki@amazon.com>

* update feedback msg (opensearch-project#5787)

Signed-off-by: Anan Z <ananzh@amazon.com>

* Resolve sort, default sort and short dot (opensearch-project#5771)

Signed-off-by: Anan Z <ananzh@amazon.com>
Signed-off-by: Anan Zhuang <ananzh@amazon.com>

* [Discover] Fixes pagination style (opensearch-project#5778)

* fixes pagination style

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* style-lint fix

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* fix react warning for unnecessary prop

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Show total hit count and pass services in embeddable

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* removed unnecessary div

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

---------

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>
Signed-off-by: Anan Zhuang <ananzh@amazon.com>
Co-authored-by: Anan Zhuang <ananzh@amazon.com>

* add changelog

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix table column error

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Fix pagination edge case

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Routing for surrounding doc link should work without a question mark appending to the end (opensearch-project#5776)

* routing for surrounding doc should work without ?

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* change path to not include appstate

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* delete optional app state for single doc link too

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

---------

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* type fixes (opensearch-project#5795)

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Update toggle to button (opensearch-project#5808)

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Adds support for line count (opensearch-project#5814)

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Add swith to datagrid table and fix ciGroup1 (opensearch-project#5816)

* add swith to datagrid table and fix ciGroup1

Signed-off-by: Anan <ananzh@amazon.com>

* fix ciGroup7

Signed-off-by: Anan Z <ananzh@amazon.com>

* fix ciGroup6

Signed-off-by: Anan Z <ananzh@amazon.com>

---------

Signed-off-by: Anan <ananzh@amazon.com>
Signed-off-by: Anan Z <ananzh@amazon.com>

* add missing snapshot (opensearch-project#5818)

Signed-off-by: Anan Z <ananzh@amazon.com>

* fix pluign ftr tests (opensearch-project#5821)

Signed-off-by: Anan Z <ananzh@amazon.com>

* update snapshot (opensearch-project#5824)

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Fix functional 3 and 4 for discover legacy (opensearch-project#5822)

* fix 3 and 4

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* fix functional 3 and 4

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* dashboard expect

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

---------

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* Solve cigroup 7 flakiness (opensearch-project#5826)

* test hover

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* comment out other ci to speed up

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* uncomment

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

---------

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>

* [Discover] Simplify feedback modal (opensearch-project#5837)

* Simplify feedback modal

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* renames datagrid  settng to newDiscover setting

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* removes sub module from branch

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

---------

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Persist line height (opensearch-project#5836)

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* Update table revert changelog (opensearch-project#5835)

* Update table revert changelog

updates changelog

<!-- List any issues this PR will resolve. Prefix the issue with the keyword closes, fixes, fix -->
<!-- Example: closes opensearch-project#1234 or fixes <Issue_URL> -->

<!-- Attach any relevant screenshots. Any change to the UI requires an attached screenshot in the PR Description -->

<!--
  Please provide detailed steps for validating your changes. This could involve specific commands to run,
  pages to visit, scenarios to try or any other information that would help reviewers verify
  the functionality of your change
-->

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

* revert dependency change

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* nit

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* update changelog

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

---------

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* fix table height persist on context page

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

---------

Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com>
Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>
Signed-off-by: Anan Z <ananzh@amazon.com>
Signed-off-by: Miki <miki@amazon.com>
Signed-off-by: Anan Zhuang <ananzh@amazon.com>
Signed-off-by: Anan <ananzh@amazon.com>
Co-authored-by: Ashwin P Chandran <ashwinpc@amazon.com>
Co-authored-by: Anan Zhuang <ananzh@amazon.com>
Co-authored-by: Miki <miki@amazon.com>
Co-authored-by: Ganesh0107 <ganesh.gopal@freshworks.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file docs Improvements or additions to documentation v2.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants