Commit Graph

5 Commits

Author SHA1 Message Date
dahoud
728b2d31d6 Revert "Fix: Define NEXT_PUBLIC variables explicitly in Dockerfile for build time"
This reverts commit e56e485fcd.
2025-10-25 17:08:34 +00:00
dahoud
e56e485fcd Fix: Define NEXT_PUBLIC variables explicitly in Dockerfile for build time
Added explicit ENV declarations for all NEXT_PUBLIC_* variables in the
builder stage before npm run build. This ensures Next.js correctly bakes
these values into the JavaScript bundle, regardless of lionsctl's
automatic manifest generation.

Variables defined:
- NEXT_PUBLIC_API_URL=https://api.lions.dev/btpxpress
- NEXT_PUBLIC_KEYCLOAK_URL=https://security.lions.dev
- NEXT_PUBLIC_KEYCLOAK_REALM=btpxpress
- NEXT_PUBLIC_KEYCLOAK_CLIENT_ID=btpxpress-frontend
- NEXT_PUBLIC_APP_ENV=production

This fixes the issue where the frontend was using https://api.lions.dev
instead of https://api.lions.dev/btpxpress for API calls.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-10-25 16:46:02 +00:00
dahoud
2cf0378270 Fix: Add NODE_ENV=production during Next.js build to load .env.production
Next.js only loads .env.production when NODE_ENV=production is set during build time.
Previously, NODE_ENV was only set in the runner stage, causing the builder stage to
ignore .env.production and use default values.

This fixes NEXT_PUBLIC_API_URL to correctly point to https://api.lions.dev/btpxpress
instead of just https://api.lions.dev.
2025-10-24 08:04:55 +00:00
dahoud
5bdd4b6e4f fix: Remove demo pages and add testDataService for production build 2025-10-02 00:10:01 +00:00
dahoud
b430bf3b96 Initial commit 2025-10-01 01:39:07 +00:00