Revert 3 production ready foodflow redesign 598c3#2190
Conversation
Key features implemented: - Removed entire enatega-multivendor-admin directory including all source files, configurations, and assets - Deleted environment configuration files (.env.dev, .env.example, .env.prod, .env.stage) - Removed development tool configurations (.eslintrc.json, .prettierrc, .npmrc, .nvmrc) - Cleaned up documentation files (README.md, LICENSE, CHANGELOG.md) - Removed all Next.js application routes and pages from the admin section - Eliminated Cypress test configurations and fixture files - Removed Firebase integration files and TypeScript configurations The changes completely remove the non-publishable admin dashboard project to resolve the publishability issue, leaving only the root configuration files.
…gn-598c3 Update from task f33b33ea-b433-465e-8b24-5923e67598c3
…gn-598c3 Update from task f33b33ea-b433-465e-8b24-5923e67598c3
…gn-598c3 Update from task f33b33ea-b433-465e-8b24-5923e67598c3
👷 Deploy request for web-loyalty-referral pending review.Visit the deploys page to approve it
|
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c67b2b8fbe
ℹ️ 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".
| SERVER_URL: process.env.NEXT_PUBLIC_SERVER_URL || "http://localhost:8001/", | ||
| WS_SERVER_URL: process.env.NEXT_PUBLIC_WS_SERVER_URL || "ws://localhost:8001/", |
There was a problem hiding this comment.
In getEnv, the STAGE and PROD branches now fall back to http://localhost:8001/ and ws://localhost:8001/ when NEXT_PUBLIC_* is unset. That makes staged/production builds silently target localhost and fail to reach the backend in any environment missing those vars (including a clean checkout after the .env.* files were removed in this commit).
Useful? React with 👍 / 👎.
| "start:web": "cd foodflow-web && npm start", | ||
| "start:admin": "cd foodflow-admin && npm start", | ||
| "install:all": "npm install", | ||
| "lint": "npm run lint --workspaces", |
There was a problem hiding this comment.
The new root lint script runs npm run lint --workspaces, but foodflow-app has no lint script, so npm run lint at the repo root fails with a missing-script error before linting can complete across the monorepo. This should use --if-present or ensure every workspace defines lint.
Useful? React with 👍 / 👎.
2 participants
Update