Skip to content

Conversation

@isabellaenriquez
Copy link
Member

Add path pattern for /checkout/

Redo pt 1 of #103069

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Nov 19, 2025
@isabellaenriquez isabellaenriquez marked this pull request as ready for review November 19, 2025 18:45
@isabellaenriquez isabellaenriquez requested a review from a team November 19, 2025 18:45
re.compile(r"^\/?(?!settings)[^/]+\/([^/]+)\/getting-started\/(.*)"),
r"/getting-started/\1/\2",
),
(re.compile(r"^\/?checkout\/[^/]+\/(.*)"), r"/checkout/\1"),
Copy link
Contributor

Choose a reason for hiding this comment

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

Bug: Checkout pattern misses paths without trailing slash

The checkout regex pattern requires a slash after the organization slug, so /checkout/acme (without trailing slash) won't match and won't be transformed, while /checkout/acme/ will transform to /checkout/. This creates inconsistent behavior. The pattern should use \/? instead of \/ before the capture group to handle both cases, similar to how the join-request pattern handles optional trailing slashes.

Fix in Cursor Fix in Web

Copy link
Member Author

Choose a reason for hiding this comment

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

the slash is required, similar to onboarding

@isabellaenriquez isabellaenriquez merged commit 6cdd208 into master Nov 19, 2025
67 checks passed
@isabellaenriquez isabellaenriquez deleted the isabella/absolute-checkout branch November 19, 2025 19:24
isabellaenriquez added a commit that referenced this pull request Nov 20, 2025
Redo pt 2 of #103069

Depends on #103668

This PR:
- Changes the new checkout route from `/checkout-v3/` to `/checkout/`
- Fixes all normalization patterns
- Adds a `/:orgId/` variant (needed for acceptance testing)
- Removes unnecessary redirect logic now that new checkout is GA
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants