egullickson/motovaultpro
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: FAB maintenance navigation (#13) #14

Merged
egullickson merged 1 commits from issue-13-fab-maintenance-nav into main 2026-01-04 04:46:19 +00:00
Conversation 0 Commits 1 Files Changed 1 +1 -2
egullickson commented 2026-01-04 04:31:38 +00:00
Owner
Copy Link

Summary

  • Fix mobile FAB 'Maintenance' option navigating to wrong screen
  • Changed navigation target from 'Vehicles' to 'Maintenance' in handleQuickAction

Fixes #13

Test Plan

  • TypeScript compilation passes
  • ESLint passes (0 errors)
  • Mobile viewport: Tap FAB (+) > Maintenance > Verify Maintenance screen displays
  • Verify maintenance forms accessible from Maintenance screen
  • No regression on other FAB actions (Log Fuel, Vehicle, Document)

Changes

File Change
frontend/src/App.tsx Updated 'add-maintenance' case to navigate to 'Maintenance' screen

🤖 Generated with Claude Code

## Summary - Fix mobile FAB 'Maintenance' option navigating to wrong screen - Changed navigation target from 'Vehicles' to 'Maintenance' in handleQuickAction Fixes #13 ## Test Plan - [x] TypeScript compilation passes - [x] ESLint passes (0 errors) - [ ] Mobile viewport: Tap FAB (+) > Maintenance > Verify Maintenance screen displays - [ ] Verify maintenance forms accessible from Maintenance screen - [ ] No regression on other FAB actions (Log Fuel, Vehicle, Document) ## Changes | File | Change | |------|--------| | `frontend/src/App.tsx` | Updated 'add-maintenance' case to navigate to 'Maintenance' screen | --- 🤖 Generated with [Claude Code](https://claude.com/claude-code)
egullickson added 1 commit 2026-01-04 04:31:39 +00:00
fix: FAB maintenance button navigates to correct screen (refs #13)
All checks were successful
Deploy to Staging / Build Images (pull_request) Successful in 5m22s
Details
Deploy to Staging / Deploy to Staging (pull_request) Successful in 37s
Details
Deploy to Staging / Verify Staging (pull_request) Successful in 6s
Details
Deploy to Staging / Notify Staging Ready (pull_request) Successful in 5s
Details
Deploy to Staging / Notify Staging Failure (pull_request) Has been skipped
Details
17484d7b5f 
The mobile FAB 'Maintenance' option was navigating to the Vehicles screen
instead of the Maintenance screen. Updated handleQuickAction to navigate
to 'Maintenance' which displays MaintenanceMobileScreen.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
egullickson merged commit 2ec208e25a into main 2026-01-04 04:46:19 +00:00
egullickson deleted branch issue-13-fab-maintenance-nav 2026-01-04 04:46:26 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
status
backlog
status/blocked
status
done
status
in-progress
status
ready
status
review
type
bug
type
chore
type
docs
type
feature
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
egullickson
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: egullickson/motovaultpro#14
Delete Branch "issue-13-fab-maintenance-nav"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?