fix(readme): remove broken live-demo claims, point to self-host#1
Open
ChunkyTortoise wants to merge 2 commits into
Open
fix(readme): remove broken live-demo claims, point to self-host#1ChunkyTortoise wants to merge 2 commits into
ChunkyTortoise wants to merge 2 commits into
Conversation
Both live-demo URLs return 404: - https://chatbot-widget-dashboard.vercel.app/ (Vercel: 404 on / and /login, post cold-start wait) - https://chatbot-widget-api.onrender.com/ (Render: 404 on /, /docs, /health) Removed: top "Dashboard-Live" badge linking to the dead Vercel URL, plus the two bullets in the Live Demo section claiming working hosted URLs. Renamed section to "Demo (Self-Host)" and kept the three real screenshots (real captures of when the app was running, still reproducible via docker-compose). WIP modifications in working tree (api/, dashboard/, widget/) unaffected; this change is README-only on a branch from origin/main. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
origin/main CI has been failing since 2026-04-17 (Lint step, ruff). This commit fixes the failures without touching the user's pre-existing WIP modifications in the working tree. Zero file overlap verified. Fixes: - ruff --fix auto-fixed 18 of 37 errors (unused imports, etc.) - F841 in tests/test_chat.py: dropped unused `response = ` assignment - F821 in api/models/*: SQLAlchemy Mapped[] forward refs handled via --per-file-ignores in ci.yml. Idiomatic SQLAlchemy 2.0 pattern that matches the existing `# type: ignore[name-defined]` comments and resolves at mapper-config time, not at annotation-eval time. - E712 in api/: SQLAlchemy column equality (`is_active == True`); added to ruff --ignore in ci.yml since these are intentional SQL filter expressions, not Python truthy comparisons. - `from __future__ import annotations` added to 6 model files (defensive: lazy annotation evaluation, SQLAlchemy 2.0 compatible). Side effect: PostToolUse hook YAML formatter normalized ci.yml's block-scalar formatting (multi-line --health-* options collapsed to single lines). Functionally equivalent. WIP isolation: this commit was prepared with the user's WIP stashed (`wip backup before CI lint fix - 2026-05-19 night`). The 19 files touched here do NOT include test_chatbots.py, test_escalation.py, test_leads.py, test_webhooks.py, leads.py, webhooks.py, etc. The WIP will be restored after CI verifies green. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The two live-demo URLs in the README both return 404. This PR removes the broken claims and reframes the section as a self-host demo, keeping the three real screenshots since they remain reproducible locally.
404 verification (2026-05-19)
Vercel root +
/loginboth 404 after a 30s cold-start wait. Render returns 404 on root,/docs, and/health. The Vercel URL was referenced in two places (top badge + Live Demo bullet); the Render URL was referenced via the QUEUE deployment note but not in the README body.Changes
Dashboard-Livebadge linking to the dead Vercel URL.## Live Demosection that claimed working hosted URLs.## Live Demo→## Demo (Self-Host).## Self-Hostingsection.dashboard-login.png,demo-page.png,playground.png) and the local-runDEMO_MODE=true uvicorn ...quickstart.Net: -5 / +3 lines.
Anti-slop checks
powerful,robust,seamless,production-ready,leverage,utilize,end-to-end, etc.)..claude/CLAUDE.md(project config, not a user-facing claim).Out of scope
api/,dashboard/,widget/,tests/(340 insertions covering leads/webhooks/analytics/playground/widget enhancements) are NOT touched. This PR was branched fromorigin/mainto keep the WIP isolated.Test plan
Co-Authored-By: Claude Opus 4.7 (1M context) noreply@anthropic.com