Skip to content

fix(frontend): remove unused router import in RoundEdit.vue#588

Merged
Jayprakash-SE merged 1 commit into
masterfrom
fix/unused-router-roundedit
Jun 26, 2026
Merged

fix(frontend): remove unused router import in RoundEdit.vue#588
Jayprakash-SE merged 1 commit into
masterfrom
fix/unused-router-roundedit

Conversation

@lgelauff

@lgelauff lgelauff commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

Dead code left by #380 (b613b20): useRouter/router in RoundEdit.vue became unused once router.reload was replaced with a data reload, but the import stayed. It trips no-unused-vars, which fails frontend lint on every branch (pre-existing on master).

Removes the two orphaned lines. eslint src/components/Round/RoundEdit.vue now clean.

Opening as draft.

@lgelauff @claude
fix(frontend): remove unused router import in RoundEdit.vue
47e3201 
`useRouter`/`router` became dead code after #380 (b613b20) replaced the
router.reload call with a data reload, but the import was left behind.
Fails `no-unused-vars`, which breaks lint on every branch.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@lgelauff lgelauff marked this pull request as ready for review June 18, 2026 08:55
@lgelauff lgelauff requested a review from Jayprakash-SE June 18, 2026 08:55
@SSpirate11 SSpirate11 mentioned this pull request Jun 26, 2026
Open
@Jayprakash-SE Jayprakash-SE merged commit 8e788a7 into master Jun 26, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Jayprakash-SE Jayprakash-SE Awaiting requested review from Jayprakash-SE

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@lgelauff @Jayprakash-SE