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

[BUG] Fix Discover table panel size auto adjust in Dashboard #5441

Merged
merged 3 commits into from
Nov 9, 2023

Conversation

ananzh
Copy link
Member

@ananzh ananzh commented Nov 7, 2023

Description

  • Add React.memo on parent comp
  • Restore embeddable panel style

Issues Resolved

#5415

dis-dashboard-bug-fix.mov

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

Copy link

codecov bot commented Nov 7, 2023

Codecov Report

Merging #5441 (7ec15e4) into main (1de8be6) will not change coverage.
Report is 2 commits behind head on main.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #5441   +/-   ##
=======================================
  Coverage   66.82%   66.82%           
=======================================
  Files        3291     3291           
  Lines       63174    63174           
  Branches    10055    10055           
=======================================
  Hits        42213    42213           
- Misses      18479    18483    +4     
+ Partials     2482     2478    -4     
Flag Coverage Δ
Linux_1 35.24% <ø> (ø)
Linux_2 55.28% <ø> (ø)
Linux_3 43.81% <ø> (+0.01%) ⬆️
Linux_4 35.34% <ø> (ø)
Windows_1 35.26% <ø> (ø)
Windows_2 55.25% <ø> (ø)
Windows_3 43.81% <ø> (ø)
Windows_4 35.34% <ø> (ø)

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

see 5 files with indirect coverage changes

@ananzh ananzh force-pushed the fix-5440-5415 branch 3 times, most recently from a10e6f8 to 7bc2815 Compare November 8, 2023 18:27
@ananzh ananzh changed the title [BUG] Fix Discover table panel size auto adjust in both Discover and Dashboard [BUG] Fix Discover table panel size auto adjust in Dashboard Nov 8, 2023
@ananzh
Copy link
Member Author

ananzh commented Nov 8, 2023

There are some flaky cypress tests. Will document here.

  • ciGroup6
Screenshot 2023-11-08 at 12 07 59

This error is because after set 2s as auto refresh interval, the superDatePicker menu panel is closed, which causes the button unable to find. This is not caused by the fix.

Screenshot 2023-11-08 at 12 09 03

* Add React.memo on parent comp
* Restore embeddable panel style

Issue Resolve
opensearch-project#5415

Signed-off-by: ananzh <[email protected]>
CHANGELOG.md Outdated Show resolved Hide resolved
ananzh and others added 2 commits November 8, 2023 22:38
Signed-off-by: ananzh <[email protected]>
Co-authored-by: Miki <[email protected]>
Signed-off-by: Anan Zhuang <[email protected]>
@ananzh ananzh merged commit e0b6006 into opensearch-project:main Nov 9, 2023
65 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Nov 9, 2023
* [BUG] Fix Discover table panel size auto adjust in Dashboard

* Add React.memo on parent comp
* Restore embeddable panel style

Issue Resolve
#5415

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

---------

Signed-off-by: ananzh <[email protected]>
Signed-off-by: Anan Zhuang <[email protected]>
Co-authored-by: Miki <[email protected]>
(cherry picked from commit e0b6006)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

# Conflicts:
#	CHANGELOG.md
opensearch-trigger-bot bot pushed a commit that referenced this pull request Nov 9, 2023
* [BUG] Fix Discover table panel size auto adjust in Dashboard

* Add React.memo on parent comp
* Restore embeddable panel style

Issue Resolve
#5415

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

---------

Signed-off-by: ananzh <[email protected]>
Signed-off-by: Anan Zhuang <[email protected]>
Co-authored-by: Miki <[email protected]>
(cherry picked from commit e0b6006)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

# Conflicts:
#	CHANGELOG.md
ananzh pushed a commit that referenced this pull request Nov 9, 2023
…5452)

* [BUG] Fix Discover table panel size auto adjust in Dashboard

* Add React.memo on parent comp
* Restore embeddable panel style

Issue Resolve
#5415

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

---------

Signed-off-by: ananzh <[email protected]>
Signed-off-by: Anan Zhuang <[email protected]>
Co-authored-by: Miki <[email protected]>
(cherry picked from commit e0b6006)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
ananzh pushed a commit that referenced this pull request Nov 9, 2023
…5453)

* [BUG] Fix Discover table panel size auto adjust in Dashboard

* Add React.memo on parent comp
* Restore embeddable panel style

Issue Resolve
#5415

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

---------

Signed-off-by: ananzh <[email protected]>
Signed-off-by: Anan Zhuang <[email protected]>
Co-authored-by: Miki <[email protected]>
(cherry picked from commit e0b6006)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 2.x bug Something isn't working discover for discover reinvent distinguished-contributor v2.11.1 Issues targeting release v2.11.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants