-
Bug
-
Resolution: Unresolved
-
Undefined
-
None
-
1.9.0
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
- Start the bulk-import workspace dev server
- Navigate to http://localhost:3000/bulk-import
- Observe redirect to http://localhost:3000/bulk-import/repositories
- Navigate to http://localhost:3000/bulk-import/repositories/repositories
- Observe the same AddRepositoriesPage component loads
- Navigate to http://localhost:3000/bulk-import/repositories/add
- Observe the same AddRepositoriesPage component loads again
- Navigate to http://localhost:3000/bulk-import/repositories/some-random-path
- 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):