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

fix: remove legacy TCO fields from vehicle forms (refs #37) #38

Merged
egullickson merged 1 commits from issue-37-remove-tco-fields into main 2026-01-16 02:11:08 +00:00
Conversation 0 Commits 1 Files Changed 5 +5 -318
egullickson commented 2026-01-15 18:39:09 +00:00
Owner
Copy Link
  • Remove CostInterval type and TCOResponse interface from frontend types
  • Remove insurance/registration cost fields from VehicleForm schema and UI
  • Keep purchasePrice and purchaseDate fields on vehicle form
  • Remove TCODisplay component from VehicleDetailPage
  • Delete TCODisplay.tsx component file
  • Remove getTCO method from vehicles API client

Legacy TCO fields moved to ownership-costs feature in #29.
Backend endpoint preserved for future reporting feature.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.5 noreply@anthropic.com

Summary

  • What does this PR change?

Linked issues

  • Fixes #
  • Relates to #

Type

  • Feature
  • Bug fix
  • Chore / refactor
  • Docs

Test plan

  • Unit tests
  • Integration tests
  • Manual verification

Commands / steps:
1.
2.

Screenshots / UI notes (if applicable)

Checklist

  • Acceptance criteria met (from linked issue)
  • No secrets committed
  • Logging is appropriate (no PII)
  • Docs updated (if needed)
- Remove CostInterval type and TCOResponse interface from frontend types - Remove insurance/registration cost fields from VehicleForm schema and UI - Keep purchasePrice and purchaseDate fields on vehicle form - Remove TCODisplay component from VehicleDetailPage - Delete TCODisplay.tsx component file - Remove getTCO method from vehicles API client Legacy TCO fields moved to ownership-costs feature in #29. Backend endpoint preserved for future reporting feature. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> ## Summary - What does this PR change? ## Linked issues - Fixes # - Relates to # ## Type - [ ] Feature - [ ] Bug fix - [ ] Chore / refactor - [ ] Docs ## Test plan - [ ] Unit tests - [ ] Integration tests - [ ] Manual verification **Commands / steps:** 1. 2. ## Screenshots / UI notes (if applicable) ## Checklist - [ ] Acceptance criteria met (from linked issue) - [ ] No secrets committed - [ ] Logging is appropriate (no PII) - [ ] Docs updated (if needed)
egullickson added 1 commit 2026-01-15 18:39:10 +00:00
fix: remove legacy TCO fields from vehicle forms (refs #37)
All checks were successful
Deploy to Staging / Build Images (pull_request) Successful in 2m42s
Details
Deploy to Staging / Deploy to Staging (pull_request) Successful in 29s
Details
Deploy to Staging / Verify Staging (pull_request) Successful in 8s
Details
Deploy to Staging / Notify Staging Ready (pull_request) Successful in 6s
Details
Deploy to Staging / Notify Staging Failure (pull_request) Has been skipped
Details
96440104c8 
- Remove CostInterval type and TCOResponse interface from frontend types
- Remove insurance/registration cost fields from VehicleForm schema and UI
- Keep purchasePrice and purchaseDate fields on vehicle form
- Remove TCODisplay component from VehicleDetailPage
- Delete TCODisplay.tsx component file
- Remove getTCO method from vehicles API client

Legacy TCO fields moved to ownership-costs feature in #29.
Backend endpoint preserved for future reporting feature.

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
egullickson merged commit 913e084127 into main 2026-01-16 02:11:08 +00:00
egullickson deleted branch issue-37-remove-tco-fields 2026-01-16 02:11:09 +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#38
Delete Branch "issue-37-remove-tco-fields"

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?