-
Notifications
You must be signed in to change notification settings - Fork 15
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 empty state to sbd details #2387
Conversation
cf5ec46
to
4707d08
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@EMaksy it looks good!
some small changes and we can merge
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @EMaksy
Some last things, but you can merge without other review from me
74bf272
to
fff75bb
Compare
fff75bb
to
196f88f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
21fd658
to
813a9cc
Compare
Description
This PR will add an empty state message for the SBD Details component in the case there are no sbd devices. Instead of just rendering the title, we render a text which informs the user.
Why?
When using disk-based SDB fencing, the SBD/fencing section in the cluster details view shows the configured SBD devices and whether they are in healthy or unhealthy status.
When using diskless SBD fencing, there are no SBD devices. Therefore, the SBD/fencing section should inform the user with a message.
Demo
Before:
After (Check Storybook):
How was this tested?
Added test and story