Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 12, 2026

When public users report issues with a name and/or email, they're shown a "Join PinPoint" CTA on the success page. However, the signup link didn't preserve their information, requiring re-entry.

Changes

Report action (src/app/report/actions.ts)

  • Add firstName, lastName, and email to success redirect URL params when new_pending=true

Success page (src/app/report/success/page.tsx)

  • Read name and email params from search params
  • Build signup URL with query string: /signup?firstName=...&lastName=...&email=...

Signup page (src/app/(auth)/signup/page.tsx)

  • Accept firstName, lastName, and email query params
  • Pre-fill form fields from params when present
  • Preserve existing invite flow (DB-sourced data takes precedence)

E2E tests (e2e/smoke/public-reporting.spec.ts)

  • Verify name-only report → success → signup pre-fill flow
  • Verify name+email report → success → signup pre-fill flow

Flow

Report with name+email → /report/success?new_pending=true&firstName=Jane&lastName=Reporter&[email protected]
                      → Click "Join PinPoint"
                      → /signup?firstName=Jane&lastName=Reporter&[email protected] (form pre-filled)

Handles all combinations: name-only, email-only, name+email, partial names (first-only, last-only), and maintains backward compatibility with invited users.

Original prompt

This section details on the original issue you should resolve

<issue_title>Name not auto-filled on signup from report success page</issue_title>
<issue_description>When a public user (new) reports an issue and provides a Name but no Email, they are directed to the success page with a 'Create Account' call to action.

However, clicking the 'Join PinPoint' link does not carry over their name to the signup page.

Technical Context:
In src/app/report/success/page.tsx, line 53:
<Link href="/signup">Join PinPoint</Link>

This link is missing query parameters (e.g., ?name=...) to pre-fill the signup form. The upstream logic redirecting to this page likely needs to pass the name as a search param so it can be forwarded to the signup route.</issue_description>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@vercel
Copy link

vercel bot commented Jan 12, 2026

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

Project Deployment Review Updated (UTC)
pin-point Ready Ready Preview, Comment Jan 12, 2026 3:04am

Copilot AI changed the title [WIP] Fix name not auto-filled on signup from report success page Pass reporter name from report success page to signup form Jan 12, 2026
@timothyfroehlich
Copy link
Owner

@copilot it should handle the email address too...

Copy link
Contributor Author

Copilot AI commented Jan 12, 2026

@copilot it should handle the email address too...

Done! Email address is now forwarded along with the name. Changes in commit 9f5903d.

Copilot AI changed the title Pass reporter name from report success page to signup form Pass reporter name and email from report success page to signup form Jan 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Name not auto-filled on signup from report success page

2 participants