Skip to content

chore: scrub live demo URLs ahead of AWS teardown - #153

Merged
Eslam93 merged 1 commit into
mainfrom
chore/scrub-live-urls
Jun 2, 2026
Merged

Eslam93 merged 1 commit into
mainfrom
chore/scrub-live-urls

Conversation

@Eslam93

@Eslam93 Eslam93 commented Jun 2, 2026

Copy link
Copy Markdown
Owner

The hosted demo is being torn down to stop AWS charges. Replace concrete URLs with placeholders / generic example domains so the public repo doesn't ship dead links.

Scope

  • README §1: live-demo table replaced with a "demo offline, here's how to reproduce" note.
  • DEVLOG: one historical mention dropped.
  • infra/DEPLOY.md and infra/bin/magnacms.ts JSDoc: `` / `` placeholders.
  • backend/app/core/config.py: same placeholder in the CORS validator error message.
  • frontend/package.json: `gen:api` default points at localhost again.
  • Two tests that asserted on the URL strings: switch to `app.example.com` / `api.example.com`.

Test plan

  • backend pytest test_localhost_cors_validator.py — 4/4 green
  • infra jest test/endpoint-context.test.ts — 25/25 green

The AWS teardown follows immediately after this merges.

The hosted demo (Amplify frontend + App Runner backend) is being torn
down to stop incurring AWS charges after the 7-day submission window.
Replace concrete URLs everywhere so the repo doesn't ship dead links:

- README §1: replace the live-demo table with a short note explaining
  the demo was taken down; point readers at infra/DEPLOY.md for the
  reproducible stack.
- DEVLOG: drop the one historical reference to the Amplify hostname in
  the 2026-05-16 entry.
- infra/DEPLOY.md and infra/bin/magnacms.ts JSDoc: replace example
  values with <your-app> / <your-apprunner-id> placeholders.
- backend/app/core/config.py: same placeholder in the CORS validator
  error message.
- frontend/package.json: gen:api default points at localhost again
  (env var override still works for anyone with a live API).
- Two tests that asserted on those URL strings: switch to the
  reserved-for-docs app.example.com / api.example.com domains.

Backend CORS tests + infra endpoint-context tests green locally.
@Eslam93
Eslam93 merged commit 7e1673a into main Jun 2, 2026
3 checks passed
@Eslam93
Eslam93 deleted the chore/scrub-live-urls branch June 2, 2026 10:10
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