-
Story
-
Resolution: Duplicate
-
Undefined
-
None
-
None
-
None
-
None
-
None
-
False
-
-
False
-
3
-
None
-
None
-
None
As part of the work on react-router v5 drop, we need to migrate from the createModalLauncher to the useModal hook instead. Before we are able to migrate to useModal, we have to refactor consumers of createModalLauncher to a hook/FC form first.
There are several such instances within KNATIVE plugin, these are listed in Inventory of createModalLauncher in Console. We should either refactor those to hook equivalent or move them under functional component. Some prior refactors that have been done are a good reference point for what is expected - https://github.com/openshift/console/pull/13506, https://github.com/openshift/console/pull/15078 or https://github.com/openshift/console/pull/15063 .
AC:
- Refactor existing uses of createModalLauncher instances by migrating them to a hook pattern
- Consolidate consumers of the new refactored hook form.
- Check the tests. Some of the refactored code could be used in tests.
- clones
-
CONSOLE-4607 [KNATIVE] Drop smaller createModalLauncher instances and use useOverlay instead.
-
- Closed
-
- is blocked by
-
CONSOLE-4607 [KNATIVE] Drop smaller createModalLauncher instances and use useOverlay instead.
-
- Closed
-