Uploaded image for project: 'OpenShift Bugs'
  1. OpenShift Bugs
  2. OCPBUGS-56245

Remove dead code for deployment and deployment config menuItems

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Unresolved
    • Icon: Undefined Undefined
    • 4.20.0
    • 4.20.0
    • Management Console
    • None
    • Quality / Stability / Reliability
    • False
    • Hide

      None

      Show
      None
    • None
    • None
    • None
    • None
    • None
    • None
    • Done
    • None
    • Hide
      Before this update, unused code in the deployment and deployment configuration menus caused unnecessary menu items to display. This update removes the unused menu item definitions, improving code maintainability and reducing potential issues in future updates. (link:https://issues.redhat.com/browse/OCPBUGS-56245[OCPBUGS-56245])
      Show
      Before this update, unused code in the deployment and deployment configuration menus caused unnecessary menu items to display. This update removes the unused menu item definitions, improving code maintainability and reducing potential issues in future updates. (link: https://issues.redhat.com/browse/OCPBUGS-56245 [ OCPBUGS-56245 ])
    • None
    • None
    • None
    • None

      Description of problem:

      As part of the drop react-router effort we are identifying instances of createModalLauncher use, which we intend to remove. Some of those instances appear to be a dead code, we should clean those up. Instances of dead code are menuItems in deployment.tsx and deployment-config.tsx. Those seem to have been superseded by the DeploymentActionFactory but not removed. We should remove those.

      Version-Release number of selected component (if applicable):

          

      How reproducible:

      always    

      Steps to Reproduce:

         This is a tech debt, no real customer facing issue occurs. Removing those should not affect the functionality of the kebab actions for deployments and deployment configs.

      Actual results:

          

      Expected results:

          

      Additional info:

          

              rh-ee-mziska Marek Ziska (Inactive)
              rh-ee-mziska Marek Ziska (Inactive)
              None
              None
              Yanping Zhang Yanping Zhang
              Jocelyn Sese Jocelyn Sese
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Created:
                Updated: