Skip to content

fix: Added SPA route catch-all#158

Merged
telivity-otaip merged 2 commits into
TelivityAI:mainfrom
Gugabit:spa-catch-all
Jul 17, 2026
Merged

fix: Added SPA route catch-all#158
telivity-otaip merged 2 commits into
TelivityAI:mainfrom
Gugabit:spa-catch-all

Conversation

@Gugabit

@Gugabit Gugabit commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Fix SPA deep-link routing for dashboard and booking engine

The Nest static-serving module defaulted to {*any}, which is interpreted as a literal route by the project’s Express 4 runtime. As a result, dashboard and booking-engine deep links were not reliably served with their SPA entry point.

Changes

  • Added an explicit renderPath: '*' for the dashboard static bundle.
  • Added an explicit renderPath: '*' for the booking-engine static bundle.
  • Replaced legacy /(.*) exclusions with path-to-regexp v8-compatible patterns:
    • /api{/*path}
    • /booking{/*path}
  • Preserved API and booking routes so they are not intercepted by the dashboard SPA fallback.

Checklist

[x] Scope is limited to api.
[x] Typecheck passed.
[x] Automated tests passed successfully.

@telivity-otaip telivity-otaip self-assigned this Jul 17, 2026
@telivity-otaip
telivity-otaip merged commit 7919d3d into TelivityAI:main Jul 17, 2026
@telivity-otaip

Copy link
Copy Markdown
Collaborator

Good stuff!

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.

2 participants