Repository navigation
Conversation
1753fb2 to
2a75d8d
Compare
brsbl
changed the title
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![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: 2a75d8d889
ℹ️ 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".
| import { SURVEY_TITLE, surveyOrder } from "../community/survey"; | ||
| import { siteHeadLinks } from "../landing/page-head"; | ||
|
|
||
| export const Route = createFileRoute("/community/survey")({ |
There was a problem hiding this comment.
This launches an end-user survey surface without corresponding SDK and bb CLI access or documentation. Repository-wide searches for the survey route and submission identifiers find only the web implementation, database storage, and web documentation; there is no SDK or CLI submission path. The root repository instructions explicitly require those surfaces to ship with every end-user UI feature, so add and document them alongside this page.
AGENTS.md reference: AGENTS.md:L26-L26
Useful? React with 👍 / 👎.
| submissionId.current ||= crypto.randomUUID(); | ||
| setSending(true); | ||
| const reply = await send(submissionId.current, surveyAnswers(values)); |
There was a problem hiding this comment.
If the server saves a response but the browser loses the acknowledgement, reloading restores the answers from localStorage but discards the submission ID held in this ref. Submitting the restored form generates another UUID, and handleSurveySubmission deduplicates only by that UUID (ON CONFLICT(id) DO NOTHING), so the same response is inserted again and counted twice. Persist and restore the ID with the pending answers, clearing both after acknowledged success, so retries after a reload remain idempotent.
Useful? React with 👍 / 👎.
a2980b7 to
e072ba6
Compare
541c6bd to
f97feb7
Compare
Unlisted /community/survey: a normal site page with the shared nav and footer, questions that adapt to earlier answers, a Please specify box for Other, progress saved in the browser, and a thank-you view that takes focus. It posts answers as JSON to /api/community/survey. BB-Thread-ID: thr_yz6t8hi7s8 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The route's head and loader pulled survey.ts into the main bundle every getbb.app page loads. The title now lives in the route, page copy lives in the page, and the loader imports the shuffle on demand. Checkbox questions read (pick any), and the error summary no longer adds a colon after a question mark. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
z.record and z.boolean pulled extra zod classes into the chunk every getbb.app page loads. The page test also follows the single-choice plugin question. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Submitting now waits on Shopify and Resend, so the button reads Sending… with a spinner and stays disabled until the reply arrives. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Section headings move to the survey model so the results email can reuse them, the two question notes sit in one table, and the sending-state and failed-save tests become one. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
f97feb7 to
316801c
Compare
Human comments
What was wrong
The survey API (#6624) had no page to fill out. Layer 2 of 4.
What changed
/community/surveypage uses the shared nav and footer,noindex, and no analytics.How you verified
BB-Thread-ID: thr_yz6t8hi7s8