Conversation
![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: a35c2bf3b4
ℹ️ About Codex in GitHub
Codex has been enabled to automatically 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 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
|
|
||
| <div id="app" | ||
| class="row g-0"></div> | ||
| {% vite_asset 'src/landing.jsx' %} |
There was a problem hiding this comment.
src/landing.jsx into the Vite manifest
This template now requires src/landing.jsx, but the Vite build config still has a single input (src/index.jsx in frontend/vite.config.ts), so production builds will not emit a manifest entry for src/landing.jsx. In production mode (config/settings/production.py sets DJANGO_VITE["default"]["dev_mode"] = False), unauthenticated users rendering this template can hit a missing-asset failure instead of loading the landing page.
Useful? React with 👍 / 👎.
| @@ -0,0 +1,63 @@ | |||
| import React from 'react' | |||
| import 'styles/Landing.css' | |||
There was a problem hiding this comment.
The component imports styles/Landing.css, but the committed stylesheet is frontend/src/styles/landing.css (lowercase). On case-sensitive Linux filesystems this import cannot resolve, which breaks bundling for the new landing entrypoint, and also for the main app entrypoint because src/index.jsx now imports this component.
Useful? React with 👍 / 👎.
1 participant
No description provided.