Skip to content

fix: remove legacy register compatibility flow - #5921

Merged
alx-xo merged 2 commits into
mainfrom
GRW-20-remove-register
Sep 4, 2026
Merged

fix: remove legacy register compatibility flow#5921
alx-xo merged 2 commits into
mainfrom
GRW-20-remove-register

Conversation

@alx-xo

@alx-xo alx-xo commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

GRW-20

Summary

Remove the legacy /register compatibility flow now that signup and zero-organization onboarding are handled by the session-aware /sign-up flow. This deletes the standalone Register component and the tests for its obsolete session, organization, assistants-disposition, and destination-routing branches.

The supported onboarding paths remain unchanged: logged-out signup and authenticated zero-organization onboarding use /sign-up, existing users use /login, and CLI onboarding sends zero-organization sessions to /sign-up with its callback destination. Slack registration, the registration API, and OAuth dynamic client registration are separate flows and are not affected.

The old path remains only as a minimal route-table redirect to /sign-up so stale links do not fail. It carries none of the removed compatibility behavior.

Motivation

The session-aware signup flow has replaced /register, and production traffic no longer depends on the legacy routing behavior. Removing it eliminates redundant onboarding logic and its dedicated test surface.

@alx-xo
alx-xo requested review from a team as code owners September 1, 2026 00:36
@alx-xo alx-xo added the bug Something isn't working label Sep 1, 2026
@linear-code

linear-code Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

GRW-20

@changeset-bot

changeset-bot Bot commented Sep 1, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 5ffe0f5

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
dashboard Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@cubic-dev-ai cubic-dev-ai Bot 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.

No issues found across 5 files

Re-trigger cubic

@alx-xo alx-xo changed the title fix: simplify legacy register redirect fix: remove legacy register compatibility flow Sep 1, 2026
@alx-xo alx-xo removed the bug Something isn't working label Sep 1, 2026
@alx-xo
alx-xo force-pushed the GRW-20-remove-register branch from 79abf46 to 5ffe0f5 Compare September 3, 2026 01:07
@alx-xo alx-xo added the preview Spawn a preview environment label Sep 3, 2026
@speakeasybot

speakeasybot commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator

🚀 Preview Environment (PR #5921)

Preview URL: https://pr-5921.dev.getgram.ai

Component Status Details Updated (UTC)
✅ Database Ready Existing database reused 2026-09-04 17:51:30.
✅ Images Available Container images ready 2026-09-04 17:51:13.

Gram Preview Bot

@alx-xo alx-xo added the review:bypass Merge without human review approval. Required status checks still apply. label Sep 4, 2026

@gram-bot gram-bot Bot 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.

Auto-approved: review:bypass label applied by @alx-xo. Required status checks still gate this merge.

@alx-xo

alx-xo commented Sep 4, 2026

Copy link
Copy Markdown
Contributor Author

Merging this as there have been no views on the /register route in the past 7 days, last view was Aug 23

image

@alx-xo
alx-xo added this pull request to the merge queue Sep 4, 2026
Merged via the queue into main with commit 825df72 Sep 4, 2026
49 checks passed
@alx-xo
alx-xo deleted the GRW-20-remove-register branch September 4, 2026 17:50
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 4, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

preview Spawn a preview environment review:bypass Merge without human review approval. Required status checks still apply.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants