Feat/mega orfeat(demo-compose): introduce demo environment for Mega UI and sync docsion demo compose#1827
Conversation
Add one-click demo stack: - docker-compose.demo.yml (+ .env.example, init-db.sh) - docs guide - minor Orion server tweaks Signed-off-by: Hongze Gao <15101764808@163.com>
Review feedback addressed: - Remove unused init-db.sh mount and delete placeholder script - Orion /v2/health now hides database error details - Add docker/demo/.env to .gitignore to prevent secret leakage - Add .dockerignore to reduce build context size - docker-compose.demo.yml: • pass CAMPSITE_RUN_MIGRATIONS to campsite_api • add sample resource limits to postgres - Docs: fix FAQ anchor, replace full-width punctuation, add hardware requirements and MySQL low-privilege note. Signed-off-by: Hongze Gao <15101764808@163.com>
- CI: web-deploy workflow builds & publishes mega-ui-demo image - Compose: • default MEGA_UI_IMAGE points to demo tag • gitmono.local domains for OAuth / CORS / Mega host • docs and health-check notes unchanged - Env: • .env.example aligned with gitmono.local + JSON CORS array • add moon/apps/web/.env.demo for APP_ENV=demo build - Docs: • update docker-compose guide (ports 9000/9001, hosts mapping section) Signed-off-by: Hongze Gao <15101764808@163.com>
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8af436b24b
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| NEXT_PUBLIC_API_URL=https://api.gitmono.local:8080 | ||
| NEXT_PUBLIC_MONO_API_URL=https://git.gitmono.local:8000 | ||
| NEXT_PUBLIC_WEB_URL=https://app.gitmono.local:3000 |
There was a problem hiding this comment.
Use HTTP for local demo endpoints
The new .env.demo bakes https://*.gitmono.local URLs into the demo UI image, but the local demo compose only exposes plain HTTP services on these ports (see docker/demo/.env.example using http://...:3000/8000/8080/8004). Because the UI Dockerfile copies .env.${APP_ENV} into .env.production at build time, the demo image will always attempt HTTPS and fail with TLS/mixed-content errors when run locally. Consider switching these demo URLs to http:// (or adding TLS termination in the demo compose) so the demo stack can actually connect.
Useful? React with 👍 / 👎.
…itmono-dev#1792) - Switch all NEXT_PUBLIC_* demo URLs from https:// to http:// to match local docker-compose (no TLS termination). - Correct typo in NEXT_PUBLIC_INTERNAL_API_URL (https://api.gitmono.local:8080 → http://api.gitmono.local:8080). Signed-off-by: Hongze Gao <15101764808@163.com>
Description
Related issue: #1792
1. Changes
CI / GitHub Actions
demoto the build matrix → publishesmega-ui-demo-0.1.0-pre-release.Docker Compose
docker/demo/docker-compose.demo.ymlMEGA_UI_IMAGEswitched to the demo tag.gitmono.local.9000(S3 API) /9001(console).moon/apps/web/.env.demoforAPP_ENV=demobuilds.Environment sample
.env.examplegitmono.localwith correct ports (3000 / 8000 / 8004 / 8080).MEGA_OAUTH__ALLOWED_CORS_ORIGINSexpressed as a JSON array.Docs
docs/demo/docker-compose-guide-for-mega-and-orion-demo-setup.md2. How to test