Skip to content

Refactor codebase - #1100

Open
veenupunyani wants to merge 2 commits into
developfrom
refactor-codebase
Open

Refactor codebase#1100
veenupunyani wants to merge 2 commits into
developfrom
refactor-codebase

Conversation

@veenupunyani

Copy link
Copy Markdown
Collaborator

Summary

  • Refactored appointment-booking after services, service-locations, login, and datetime: shared BookingDetailCallout and SearchRow (optional nearest pin), split useNearestSort, and removed duplicate search components.
  • Collapsed location types into a single ServiceLocation in api/service-locations.ts and deleted unused api/locations.ts / getBookingLocations.
  • Updated dependencies: React Router 8.3, license-checker-rseidelsohn v5 (clears prior audit noise, this one is much better updated); aligned GitHub CI with Dockerfile on Node 24; fixed Vite ~/ alias for ESM.

Test plan

  • npm ci and npm run ci:check on Node 24
  • npm audit shows 0 vulnerabilities

Shared BookingDetailCallout and SearchRow, collapsed ServiceLocation into one type, bumped React Router to 8.3 and CI/Docker-aligned Node to 24, and replaced license-checker with license-checker-rseidelsohn.
@sonarqubecloud

Copy link
Copy Markdown

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.

1 participant