Skip to content

Stabilize Netlify deployment for Codex Solar site#7

Open
attaboy11 wants to merge 2 commits into
mainfrom
codex/initialize-codex-solar-project-6gkj6k
Open

Stabilize Netlify deployment for Codex Solar site#7
attaboy11 wants to merge 2 commits into
mainfrom
codex/initialize-codex-solar-project-6gkj6k

Conversation

@attaboy11
Copy link
Copy Markdown
Owner

Summary

  • remove the legacy aigiftgen_v101 2 snapshot so the repository contains a single Next.js app
  • configure Next.js for static export and add a Netlify build manifest that matches the platform defaults
  • document the Netlify deployment flow and ensure metadataBase is set to avoid build warnings

Testing

  • CI=1 npm run build

Codex Task

@vercel
Copy link
Copy Markdown

vercel Bot commented Nov 13, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
aigiftgen Error Error Nov 14, 2025 11:25am
aigiftgen-qtxz Ready Ready Preview Comment Nov 14, 2025 11:25am

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ 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".

Comment thread app/page.tsx
Comment on lines +220 to +229
<form className="grid gap-4 text-sm text-slate-900">
<input type="text" placeholder="Name" className="bg-white/90 text-slate-900" />
<input type="email" placeholder="Work email" className="bg-white/90 text-slate-900" />
<textarea
placeholder="Tell us about your buildings, timelines and energy goals."
rows={4}
className="w-full rounded-2xl border border-white/30 bg-white/90 px-4 py-3 text-slate-900 outline-none focus:border-emerald-400 focus:ring-2 focus:ring-emerald-400/60"
/>
<button type="submit" className="btn-primary">
Send briefing
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Contact form never submits any data

The new lead form renders visually but it cannot send anything: the <form> has no action/onSubmit handler and, more critically, none of the inputs/textarea are given a name. When the user clicks “Send briefing” the browser therefore performs a GET back to / with an empty query string, so Netlify receives no lead details and the visitor sees a page refresh only. This makes the primary CTA of the page inoperative and should be fixed by wiring the form to Netlify or a backend and giving every field a name so the values can be transmitted.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant