Replace root path and absolute path navigation with named routes
- Dominant language
- JavaScript
- Stars
- 400
- Forks
- 89
- Avg merge
- 1d 21h
- Merged PRs (30d)
- 146
Description
Part of #8094 (prerequisite: #8084)
## What
14 places use hardcoded absolute path strings for navigation instead of named routes.
### Root path pushes: `router.push('/')` -> `{ name: 'home' }`
| File | Line |
|------|------|
| `pages/account/Create.vue` | 264 |
| `pages/admin/index.vue` | 23 |
| `pages/admin/AuditLog.vue` | 50 |
| `pages/admin/Overview.vue` | 132 |
| `pages/admin/Teams.vue` | 197 |
| `pages/admin/Users/General.vue` | 124 |
| `pages/admin/Settings/General.vue` | 644 |
| `pages/admin/Settings/General.vue` | 679 |
| `pages/team/registerDevice.vue` | 39 |
### Absolute path objects: `{ path: '/admin/...' }` -> `{ name: '...' }`
| File | Line | Current | Should be |
|------|------|---------|-----------|
| `pages/admin/Users/createUser.vue` | 151 | `{ path: '/admin/users' }` | `{ name: 'admin-users' }` |
| `pages/admin/Users/UserDetails.vue` | 146 | `{ path: '/admin/users' }` | `{ name: 'admin-users' }` |
| `pages/admin/Settings/SSO/index.vue` | 64 | `{ path: '/admin/settings' }` | `{ name: 'admin-settings' }` |
| `pages/admin/Settings/SSO/createEditProvider.vue` | 266 | `{ path: '/admin/settings' }` | `{ name: 'admin-settings' }` |
| `pages/admin/Template/index.vue` | 6 | `:to="{path: '/admin/templates'}"` | `:to="{ name: 'admin-templates' }"` |
All straightforward find-and-replace.
Contributor guide
Research direction
Start with the 14 locations listed in the issue across the pages/ directory and inspect the surrounding router.push and link bindings. Replace each root-path or absolute-path navigation with the named route shown in the tables. Done means all listed locations use the specified route names and no listed hardcoded navigation paths remain.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- frontend
- Issue type
- Refactor
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Quiet
- Clarity
- Clearly specified
- Newbie friendliness
- 84/100