Uploaded image for project: 'Red Hat Developer Hub Bugs'
  1. Red Hat Developer Hub Bugs
  2. RHDHBUGS-2531

Clean up bulk-import plugin routes - remove duplicate paths loading same component

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Unresolved
    • Icon: Undefined Undefined
    • None
    • 1.9.0
    • Bulk Import

      Description of problem:

      The bulk-import plugin currently defines multiple routes that all load the same
      AddRepositoriesPage component:

      • /bulk-import → redirects to /bulk-import/repositories
      • /bulk-import/repositories/ → AddRepositoriesPage
      • /bulk-import/repositories/repositories → AddRepositoriesPage
      • /bulk-import/repositories/add → AddRepositoriesPage

      This creates unnecessary complexity and confusion. Additionally, there is no handling for undefined paths under /bulk-import/*, which could lead to blank pages.

      Prerequisites (if any, like setup, operators/versions):

      Steps to Reproduce

      1. Start the bulk-import workspace dev server
      2. Navigate to http://localhost:3000/bulk-import
      3. Observe redirect to http://localhost:3000/bulk-import/repositories
      4. Navigate to http://localhost:3000/bulk-import/repositories/repositories
      5. Observe the same AddRepositoriesPage component loads
      6. Navigate to http://localhost:3000/bulk-import/repositories/add
      7. Observe the same AddRepositoriesPage component loads again
      8. Navigate to http://localhost:3000/bulk-import/repositories/some-random-path
      9. Observe the Blank page appears

        Actual results:

      • Three different URLs load the identical AddRepositoriesPage component
      • Undefined paths under /bulk-import/* are not handled (no redirect)
      • Route configuration is unnecessarily complex with redundant entries
      • addRepositoriesRouteRef is exported but serves no unique purpose

        Expected results:

      • Single /bulk-import path loads AddRepositoriesPage
      • /bulk-import/import-history/:repoUrl loads ImportHistoryPage
      • Any undefined paths under /bulk-import/* redirect to /bulk-import
      • Clean route configuration without duplicate entries
      • Remove unused addRepositoriesRouteRef from exports

        Reproducibility (Always/Intermittent/Only Once):

        Build Details:

        Additional info (Such as Logs, Screenshots, etc):

              rh-ee-mitkumar Mitesh Kumar
              rh-ee-mitkumar Mitesh Kumar
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated: