Conditionally render model registry modals #3304
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Towards RHOAIENG-12117
Description
Update usage of modals in the model registry section to render them only when they would be shown rather than having them be rendered but hidden.
How Has This Been Tested?
Run thru the UI and check that then modals are shown only when they should be shown.
Model Registry
pageArchive model
Archive model?
modal is shownModel Registry
pageDeploy
Deploy model
modal is shownModel Registry
pageArchive model version
Archive version?
modal is shownModel Registry
pageDeploy
Deploy model
modal is shownModel Registry
pageArchive model
Archive version?
modal is shownModel Registry
pageRegister model
button, selectView archived models
Restore version
Restore version
modal is shownModel Registry
pageRegister model
button, selectView archived models
Restore model
Restore model
modal is shownModel Serving
page (titledDeployed models
)Deploy model
buttonDeploy model
modal is shownModel Serving
page (titledDeployed models
)Edit
Edit model
modal is shownTest Impact
Current tests already test these modals.
Request review criteria:
Self checklist (all need to be checked):