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
Remove Edit Event Definition button from events scoped as immutable #16515
Remove Edit Event Definition button from events scoped as immutable #16515
Conversation
…-remove-or-edit-notification-in-illuminate-created-event-in-51x
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.
Looks good and tests successfully!
Nice call opening up to the edit page of the event definition copy when the duplicate button is clicked, makes the UX very seamless.
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 and tests successfully too!
…-remove-or-edit-notification-in-illuminate-created-event-in-51x
…ation-in-illuminate-created-event-in-51x
…ation-in-illuminate-created-event-in-51x
@@ -104,6 +133,14 @@ const ViewEventDefinitionPage = () => { | |||
notifications={notifications} /> | |||
</Col> | |||
</Row> | |||
{showDialog && ( |
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.
Looks like the dialog is creating some sort of race condition where is trying to update the state of a component that isn't on the DOM. To solve the issue, move this block outside the DocumentTitle block and wrap both components in a fragment.
<>
<DocumentTitle ....>
</DocumentTitle>
{showDialog && (
<ConfirmDialog ....>
</ConfirmDialog>
)}
</>
…16515) * Remove Edit Event Definition button from events scoped as immutable * Add duplicate button on View Event Definition page
Description
Edit Event Definition
button if the event definition is scoped as immutable. The edit option is already hidden from the More Actions menu.Duplicate Event Definition
button to the ViewEventDefinition page toolbarMotivation and Context
closes #15916
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: