Ensure sponsor logos use light background#26
Conversation
There was a problem hiding this comment.
💡 Codex Review
Lines 8 to 18 in af50a0a
The commit creates a dedicated Sponsors route component but the central router configuration still only exposes /program, /organizers, /sponsorship, and /contact. Without adding /sponsors here, the new component is never imported or rendered, so the intended white background for logos never affects the application and the page is unreachable.
ℹ️ 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
| import { Button } from "../../components/ui/button"; | ||
| import { Card, CardContent, CardFooter } from "../../components/ui/card"; | ||
| import sponsorsData from "../../data/sponsors.json"; | ||
|
|
There was a problem hiding this comment.
Import sponsors data file that doesn’t exist
The new Sponsors route imports ../../data/sponsors.json, but that file isn’t present anywhere in the repository. As soon as this component is wired into the router, the build will fail with a module‑not‑found error and the page cannot render. Either add the JSON file or reuse existing data before enabling the route.
Useful? React with 👍 / 👎.
Summary