Skip to content

feat: rename Device and Instance parent route names to kebab-case - #8191

Merged
cstns merged 2 commits into
mainfrom
feat/standardize-route-names/8189-device-instance
Aug 14, 2026
Merged

feat: rename Device and Instance parent route names to kebab-case#8191
cstns merged 2 commits into
mainfrom
feat/standardize-route-names/8189-device-instance

Conversation

@cstns

@cstns cstns commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Closes #8189
Part of #8084

Summary

  • Renames Device to device and Instance to instance in their route definitions
  • Updates all 29 references across 21 files
  • These two parent route names were missed from the sub-issue breakdown, all their child routes were already kebab-case

Test plan

🤖 Generated with Claude Code

These two PascalCase parent route names were missed from the route
rename effort. All child routes were already kebab-case.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@cstns cstns self-assigned this Aug 14, 2026
@cstns
cstns requested a review from n-lark August 14, 2026 10:01
@cstns
cstns deployed to staging August 14, 2026 10:04 — with GitHub Actions Active
@codecov

codecov Bot commented Aug 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.16%. Comparing base (a24ab9a) to head (916140b).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8191      +/-   ##
==========================================
- Coverage   76.19%   76.16%   -0.04%     
==========================================
  Files         442      442              
  Lines       23678    23694      +16     
  Branches     6299     6306       +7     
==========================================
+ Hits        18042    18047       +5     
- Misses       5636     5647      +11     
Flag Coverage Δ
backend 76.16% <ø> (-0.04%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@n-lark n-lark left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I pushed a fix for a missed Instance -> instance in ChangeInstanceType - otherwise looks good!

  • Grep for old names as quoted strings confirms no remaining references
  • Navigation to device and instance pages works as before

@cstns
cstns merged commit 4fe037c into main Aug 14, 2026
28 of 29 checks passed
@cstns
cstns deleted the feat/standardize-route-names/8189-device-instance branch August 14, 2026 13:27
@cstns
cstns deployed to staging August 14, 2026 13:28 — with GitHub Actions Active
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Rename Device and Instance parent route names to kebab-case

2 participants