Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support optional segments in check routes CLI command #774

Merged
merged 3 commits into from
Apr 18, 2023

Conversation

frandiox
Copy link
Contributor

Closes #749

The h2 check routes command was matching ($lang)/cart but not (lang)/cart (i.e. optional segment instead of optional parameter). I've added support for that and also for optional segments and parameters in any position of the URL. It will now also match /cart/(extra) since the extra bit is optional.

@frandiox frandiox requested a review from a team April 11, 2023 10:07
@github-actions

This comment has been minimized.

Copy link
Contributor

@cartogram cartogram left a comment

Choose a reason for hiding this comment

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

Nice work Fran! 🥇

@frandiox frandiox merged commit 2039a4a into 2023-01 Apr 18, 2023
@frandiox frandiox deleted the fd-check-routes-optional-segments branch April 18, 2023 05:47
This was referenced Apr 18, 2023
lordofthecactus added a commit that referenced this pull request Apr 20, 2023
lordofthecactus added a commit that referenced this pull request Apr 20, 2023
* Revert "Support optional segments in `check routes` CLI command (#774)"

This reverts commit 2039a4a.

* update package.lock
blittle pushed a commit that referenced this pull request May 12, 2023
* Revert "Support optional segments in `check routes` CLI command (#774)"

This reverts commit 2039a4a.

* update package.lock
frehner added a commit that referenced this pull request May 17, 2023
* Fix 2023 01 (#807)

* Revert "Support optional segments in `check routes` CLI command (#774)"

This reverts commit 2039a4a.

* update package.lock

* Fix demo store search action url (#802)

* Fixing #825

* Add loading dependency to redirect effect

* Revert "Fix 2023 01 (#807)"

This reverts commit 53e6dbf.

* Revert "Fix demo store search action url (#802)"

This reverts commit a783b42.

* add a changeset

---------

Co-authored-by: Daniel Rios <[email protected]>
Co-authored-by: Ós <[email protected]>
Co-authored-by: Anthony Frehner <[email protected]>
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.

check routes incompatible with optional path segments and pathless layout routes
2 participants