Skip to content

fix(signup): drop Standard profile absence copy - #181

Merged
lukedanielchapman merged 1 commit into
mainfrom
fix/signup-no-absence-copy
Aug 2, 2026
Merged

fix(signup): drop Standard profile absence copy#181
lukedanielchapman merged 1 commit into
mainfrom
fix/signup-no-absence-copy

Conversation

@lukedanielchapman

Copy link
Copy Markdown
Contributor

Summary

  • Remove “You’re all set for chat — no AI key needed…” from Standard Profile
  • Standard still shows username only; BYOK still shows the key fields

Test plan

  • Standard register Profile: username + continue; no “no AI key” note
  • BYOK Profile still asks for AI key

Say what the step is for. Do not narrate that an AI key field is missing.
Copilot AI review requested due to automatic review settings August 2, 2026 21:34
@vercel

vercel Bot commented Aug 2, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
atom-registry Ready Ready Preview Aug 2, 2026 9:34pm

Request Review

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Removes the “no AI key needed” informational note from the hosted Standard registration flow’s Profile step, while preserving the hosted BYOK path that still collects AI key details.

Changes:

  • Deleted the Standard-lane Profile note copy shown during hosted registration.
  • Updated the Profile step conditional rendering so hosted registration only shows BYOK key fields (and shows nothing extra for hosted Standard).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@lukedanielchapman
lukedanielchapman merged commit ad61333 into main Aug 2, 2026
4 checks passed
@lukedanielchapman
lukedanielchapman deleted the fix/signup-no-absence-copy branch August 2, 2026 21:37
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.

2 participants