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

fix: Wire up Add Maintenance button on vehicle detail page (#194) #195

Merged
egullickson merged 1 commits from issue-194-fix-add-maintenance-button into main 2026-02-15 16:09:53 +00:00
Conversation 0 Commits 1 Files Changed 3
+69 -7
egullickson commented 2026-02-15 16:01:52 +00:00
Owner
Copy Link
Copy Source

Fixes #194

Summary

  • Renamed "Schedule Maintenance" to "Add Maintenance" on desktop and mobile vehicle detail pages
  • Changed button style to variant="contained" on desktop to match "Add Fuel Log"
  • Wired button to open an inline dialog with MaintenanceRecordForm (same pattern as Add Fuel Log)
  • Added onSuccess callback prop to MaintenanceRecordForm for dialog close + query invalidation
  • Mobile dialog opens fullscreen; desktop dialog is responsive with fullScreen at small breakpoints

Files Changed

  • frontend/src/features/vehicles/pages/VehicleDetailPage.tsx - Desktop button + dialog
  • frontend/src/features/vehicles/mobile/VehicleDetailMobile.tsx - Mobile button + dialog
  • frontend/src/features/maintenance/components/MaintenanceRecordForm.tsx - Added onSuccess prop

Test Plan

  • Click "Add Maintenance" on desktop vehicle detail page - dialog opens with form
  • Submit maintenance record - dialog closes, toast confirms success
  • Click "Add Maintenance" on mobile vehicle detail page - fullscreen dialog opens
  • Submit maintenance record on mobile - dialog closes, toast confirms success
  • Verify "Add Maintenance" button matches "Add Fuel Log" styling on desktop
Fixes #194 ## Summary - Renamed "Schedule Maintenance" to "Add Maintenance" on desktop and mobile vehicle detail pages - Changed button style to `variant="contained"` on desktop to match "Add Fuel Log" - Wired button to open an inline dialog with `MaintenanceRecordForm` (same pattern as Add Fuel Log) - Added `onSuccess` callback prop to `MaintenanceRecordForm` for dialog close + query invalidation - Mobile dialog opens fullscreen; desktop dialog is responsive with `fullScreen` at small breakpoints ## Files Changed - `frontend/src/features/vehicles/pages/VehicleDetailPage.tsx` - Desktop button + dialog - `frontend/src/features/vehicles/mobile/VehicleDetailMobile.tsx` - Mobile button + dialog - `frontend/src/features/maintenance/components/MaintenanceRecordForm.tsx` - Added `onSuccess` prop ## Test Plan - [ ] Click "Add Maintenance" on desktop vehicle detail page - dialog opens with form - [ ] Submit maintenance record - dialog closes, toast confirms success - [ ] Click "Add Maintenance" on mobile vehicle detail page - fullscreen dialog opens - [ ] Submit maintenance record on mobile - dialog closes, toast confirms success - [ ] Verify "Add Maintenance" button matches "Add Fuel Log" styling on desktop
egullickson added 1 commit 2026-02-15 16:01:53 +00:00
fix: wire up Add Maintenance button on vehicle detail page (refs #194)
Deploy to Staging / Build Images (pull_request) Successful in 3m24s
Details
Deploy to Staging / Deploy to Staging (pull_request) Successful in 52s
Details
Deploy to Staging / Verify Staging (pull_request) Successful in 8s
Details
Deploy to Staging / Notify Staging Ready (pull_request) Successful in 7s
Details
Deploy to Staging / Notify Staging Failure (pull_request) Has been skipped
Details
7140c7e8d4 
Rename "Schedule Maintenance" to "Add Maintenance", match contained
button style to "Add Fuel Log", and open inline MaintenanceRecordForm
dialog on click. Applied to both desktop and mobile views.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
egullickson merged commit 963c17014c into main 2026-02-15 16:09:53 +00:00
egullickson deleted branch issue-194-fix-add-maintenance-button 2026-02-15 16:09:53 +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 labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
egullickson
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: egullickson/motovaultpro#195