Add FedEx WebHarbor mirror#50
Conversation
Review — Add FedEx WebHarbor mirror (#50)Verdict: Request changes (one blocker), then Approve. The code is correct and the full Mechanical
Visual fidelity: PASSCorrect FedEx purple branding, nav, and a full tracking timeline (screenshot: Functional depth: PASSTracking shows the delay reason; support search resolves the exception article; Task quality: 18/18 PASSAll entities verified present (5 tracking numbers, 4 seeded users, named locations/articles). Required before approval: upload the FedEx asset tarball to HF + bump |
Summary
Implementation summary
This PR adds a self-contained FedEx-style logistics mirror under
sites/fedex/with Flask, SQLAlchemy, SQLite, Jinja templates, local CSS/JS, deterministic SVG visuals, seeded tracking timelines, rate estimates, shipment creation, pickup scheduling, invoices, claims, support content, and account flows.Selected WebSyn port:
40015Seed row counts
Benchmark users
TestPass123!Task count and categories
sites/fedex/tasks.jsonlKey routes
/,/home/track,/track/results,/tracking/<tracking_number>/rate-estimate/ship,/ship/service,/ship/review,/ship/confirmation/pickup/locations,/locations/<location_slug>/support,/support/<article_slug>/search/login,/register,/logout/account,/account/edit,/account/shipments,/invoices,/claimsVerification commands and results
py -3 -m py_compile sites/fedex/app.py sites/fedex/seed_data.py sites/fedex/_health.py-> passedpy -3 sites/fedex/seed_data.py-> generated deterministicinstance_seed/fedex.dband local SVG assetsDocker result
docker build -t webharbor:fedex-pr .-> passedwebharbor-fedex-submit8301 -> 8101,43000-43015 -> 40000-40015Smoke test result
200on/,/home,/track,/track/results,/rate-estimate,/locations,/support,/search?q=delay,/loginSH-260061, trackingFDX260000061, pickupPU-2609Reset + MD5 result
POST http://localhost:8301/reset/fedex->ready: truedocker exec webharbor-fedex-submit md5sum /opt/WebSyn/fedex/instance/fedex.db /opt/WebSyn/fedex/instance_seed/fedex.db-> both hashes matchedea65ea7d0ec6f17af8229806ed8c4317200Screenshot paths
Local review-only screenshots were captured at:
sites/fedex/scraped_data/submission_review/home.pngsites/fedex/scraped_data/submission_review/track.pngsites/fedex/scraped_data/submission_review/track_results.pngsites/fedex/scraped_data/submission_review/tracking_detail.pngsites/fedex/scraped_data/submission_review/rate_estimate.pngsites/fedex/scraped_data/submission_review/locations.pngsites/fedex/scraped_data/submission_review/support.pngsites/fedex/scraped_data/submission_review/login.pngsites/fedex/scraped_data/submission_review/mobile_home.pngHF asset status
hf upload --create-pr:.assets-revisionwas not modified in this PR and should stay unchanged until the HF asset change is merged/pinned.Known limitations
scripts/check_assets.sh/scripts/fetch_assets.shuse CRLF-sensitive bash entrypoints in this Windows checkout, so I followed the documented asset workflow with equivalent manualhf/tarcommands in PowerShell when needed.Manual review notes