-
Notifications
You must be signed in to change notification settings - Fork 305
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
feat(metadata-sidebar): disable delete button #3677
Conversation
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.
we need to add unit tests for when the button is disabled under specific conditions and enabled under different conditions
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.
we can also add a storybook story to showcase that the button is disabled
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.
added both unit and storybook tests
01410b9
to
081b2af
Compare
081b2af
to
35a9a98
Compare
* feat(metadata-sidebar): disable delete button * feat(metadata-sidebar): remove console.log * feat(metadata-sidebar): unit tests * feat(metadata-sidebar): storybook tests
This pull request disables the Delete button in the MetadataInstanceEditor when a user starts creating a new template. Since there's nothing to delete before the instance is created, the button stays inactive.
Editing new template chosen from dropdown:
Editing already existing instance: