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

Add Notification Modal on Indices page #1143

Merged

Conversation

CaptainDredge
Copy link
Contributor

@CaptainDredge CaptainDredge commented Aug 22, 2024

Description

This will get merged only after #1141 as its a dependent PR

Screenshot 2024-08-22 at 10 23 55 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
Contributor

@skumawat2025 skumawat2025 left a comment

Choose a reason for hiding this comment

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

LGTM.
Just one comment.
We might want to fix the yellow line that's coming in this screen and wants to have the unsaved changes in aligned with buttons.

image

@CaptainDredge CaptainDredge force-pushed the notification-on-newChanges branch 2 times, most recently from 368951c to 4a5f3d6 Compare August 22, 2024 15:18
Signed-off-by: Prabhat Sharma <ptsharma@amazon.com>
@CaptainDredge
Copy link
Contributor Author

CaptainDredge commented Aug 22, 2024

@skumawat2025 updated the bottom buttons, please take a look at the screenshot in issue description

Copy link
Contributor

@skumawat2025 skumawat2025 left a comment

Choose a reason for hiding this comment

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

LGTM.

@SuZhou-Joe SuZhou-Joe merged commit 620551b into opensearch-project:main Aug 23, 2024
12 of 13 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Aug 23, 2024
Signed-off-by: Prabhat Sharma <ptsharma@amazon.com>
Co-authored-by: Prabhat Sharma <ptsharma@amazon.com>
(cherry picked from commit 620551b)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
SuZhou-Joe pushed a commit that referenced this pull request Aug 23, 2024
(cherry picked from commit 620551b)

Signed-off-by: Prabhat Sharma <ptsharma@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: Prabhat Sharma <ptsharma@amazon.com>
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.

3 participants