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

Use TopNavControlButtonData for channel test button #243

Merged
merged 1 commit into from
Aug 30, 2024

Conversation

AMoo-Miki
Copy link
Contributor

Description

Use TopNavControlButtonData for channel test button

Note: All other buttons and the general styling of the page are created wrongly and with custom styling so nothing matches.

Screenshot 2024-08-23 at 1 47 33 PM

Check List

  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Copy link

codecov bot commented Aug 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.79%. Comparing base (0bad488) to head (e8ad499).
Report is 8 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #243   +/-   ##
=======================================
  Coverage   82.79%   82.79%           
=======================================
  Files          55       55           
  Lines        1686     1686           
  Branches      455      455           
=======================================
  Hits         1396     1396           
  Misses        283      283           
  Partials        7        7           
Flag Coverage Δ
dashboards-notifications 82.79% <ø> (ø)

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.

@amsiglan amsiglan merged commit 6121bd4 into opensearch-project:main Aug 30, 2024
9 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Aug 30, 2024
Signed-off-by: Miki <[email protected]>
(cherry picked from commit 6121bd4)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Hailong-am pushed a commit that referenced this pull request Sep 4, 2024
(cherry picked from commit 6121bd4)

Signed-off-by: Miki <[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>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Sep 17, 2024
Signed-off-by: Miki <[email protected]>
(cherry picked from commit 6121bd4)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
AWSHurneyt pushed a commit that referenced this pull request Sep 17, 2024
(cherry picked from commit 6121bd4)

Signed-off-by: Miki <[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>
jowg-amazon added a commit to jowg-amazon/dashboards-notifications that referenced this pull request Sep 24, 2024
jowg-amazon added a commit to jowg-amazon/dashboards-notifications that referenced this pull request Sep 24, 2024
AWSHurneyt pushed a commit that referenced this pull request Sep 24, 2024
* Revert "[Backport 2.17] Fit and Finish UX Fixes Pt 2. (#275)"

This reverts commit 5ac0919.

Signed-off-by: Joanne Wang <[email protected]>

* Revert "Fit and Finish UX Fixes (#263) (#269)"

This reverts commit b3dd31c.

Signed-off-by: Joanne Wang <[email protected]>

* Revert "Use TopNavControlButtonData for channel test button (#243) (#268)"

This reverts commit 127eae4.

Signed-off-by: Joanne Wang <[email protected]>

* Revert "[Fit & Finish] Updated Fit and Finish guidelines (#256) (#259)"

This reverts commit fb48dbc.

Signed-off-by: Joanne Wang <[email protected]>

---------

Signed-off-by: Joanne Wang <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants