Skip to content

feat(auth): pre-fill email address in org invite registration - #122473

Open
mayuriphad wants to merge 1 commit into
getsentry:masterfrom
mayuriphad:fix-org-invite-email
Open

feat(auth): pre-fill email address in org invite registration#122473
mayuriphad wants to merge 1 commit into
getsentry:masterfrom
mayuriphad:fix-org-invite-email

Conversation

@mayuriphad

Copy link
Copy Markdown

Resolves #121594 by properly passing \invite_email\ to the frontend context and pre-filling it on the registration form.

Copilot AI lite review requested due to automatic review settings August 24, 2026 16:16
@mayuriphad
mayuriphad requested review from a team as code owners August 24, 2026 16:16

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions github-actions Bot added Scope: Frontend Automatically applied to PRs that change frontend components Scope: Backend Automatically applied to PRs that change backend components labels Aug 24, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🚨 Warning: This pull request contains Frontend and Backend changes!

It's discouraged to make changes to Sentry's Frontend and Backend in a single pull request. The Frontend and Backend are not atomically deployed. If the changes are interdependent of each other, they must be separated into two pull requests and be made forward or backwards compatible, such that the Backend or Frontend can be safely deployed independently.

Have questions? Please ask in the #discuss-dev-infra channel.

@Laithalrubaie

Laithalrubaie commented Aug 24, 2026

Copy link
Copy Markdown

nice, thanks for picking this up! just a heads up on scope — this touches static/app/views/auth/* (registerForm.tsx, login.tsx), which is only reachable via experimentalSpaRoutes in router/routes.tsx, gated behind EXPERIMENTAL_SPA. that flag's only ever set for pnpm run dev-ui (frontend-only local dev), so it doesn't run on real sentry.io. prod's /auth/register/ is still server-rendered by AuthLoginView + sentry/login.html, so this diff won't reach actual users yet.

also, since it's just initialData.username, the field stays fully editable rather than locked to the invite — might be worth tightening up too, no big deal either way.

I put up a fix for the prod path here if it's useful as a reference: #122430 (+ #122431 for the accept-invite screen itself). Happy to help however's useful!

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

Labels

Scope: Backend Automatically applied to PRs that change backend components Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Registration form in organization invite flow should pre-fill email address

3 participants