FlowFuse / FlowFuse/flowfuse

Replace root path and absolute path navigation with named routes

Open Beginner friendly
#8,096 0 comments 0 reactions 0 assignees View on GitHub
area:frontend size:S type:enhancement
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.