sb/ACC-System
1
0
Fork 0
Code Issues Pull requests Projects Releases 2 Packages Wiki Activity Actions

Remove PDF booking-sheet import and dead code #2

Merged
sb merged 1 commit from cleanup into main 2026-08-08 19:42:07 +10:00
Conversation 0 Commits 1 Files changed 19 +32 -1387
sb commented 2026-08-08 19:33:50 +10:00
Owner
Copy link

The "Import from Booking Sheet" PDF pre-fill on the new booking page is no longer needed now that all historical bookings have been migrated. Removed PdfImportButton, the parse-pdf route, and its parser-only helpers (kept the two shared date-display helpers still used elsewhere).

Also cleaned up while in there:

  • An orphaned BookingRow component (never imported anywhere)
  • A handful of unused exported functions/types across src/lib
  • The unused pdf-parse and pdf2pic npm packages — neither was actually imported anywhere, even by the deleted route, which shelled out to ghostscript directly
  • Renamed vitest.config.ts to .mts to fix a Vite config-loader ESM/CommonJS warning

Verified: build clean, lint clean (0 errors, 0 unused eslint-disable directives via --report-unused-disable-directives), 174/174 tests passing, and confirmed live in a running dev server that the import button is gone from /bookings/new while the rest of the form still renders correctly.

The "Import from Booking Sheet" PDF pre-fill on the new booking page is no longer needed now that all historical bookings have been migrated. Removed `PdfImportButton`, the `parse-pdf` route, and its parser-only helpers (kept the two shared date-display helpers still used elsewhere). Also cleaned up while in there: - An orphaned `BookingRow` component (never imported anywhere) - A handful of unused exported functions/types across `src/lib` - The unused `pdf-parse` and `pdf2pic` npm packages — neither was actually imported anywhere, even by the deleted route, which shelled out to `ghostscript` directly - Renamed `vitest.config.ts` to `.mts` to fix a Vite config-loader ESM/CommonJS warning Verified: build clean, lint clean (0 errors, 0 unused `eslint-disable` directives via `--report-unused-disable-directives`), 174/174 tests passing, and confirmed live in a running dev server that the import button is gone from `/bookings/new` while the rest of the form still renders correctly.
The "Import from Booking Sheet" PDF pre-fill on the new booking page is
no longer needed now that all historical bookings have been migrated.
Removed PdfImportButton, the parse-pdf route, and its parser-only
helpers.

Also cleaned up while in there: an orphaned BookingRow component, a
handful of unused exported functions/types, the unused pdf-parse and
pdf2pic npm packages (never actually imported, even by the deleted
route), and renamed vitest.config.ts to .mts to fix a Vite
config-loader warning.
sb merged commit f143eb31d0 into main 2026-08-08 19:42:07 +10:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
sb/ACC-System!2
No description provided.