Skip to content

Conversation

fjaguero
Copy link
Contributor

@fjaguero fjaguero commented Oct 9, 2015

Yesterday I spent a lot of time to figure out how to do this. In fact, I found like 4-5 search results regarding the same issue. There is no place that actually tells you that you can do something like "/:user?/brands/:brand?/home" using this syntax.

Yesterday I spent a lot of time to figure out how to do this. In fact, I found like 4-5 search results regarding the same issue. There is no place that actually tells you that you can do something like "/:user?/brands/:brand?/home" using this syntax.
knowbody added a commit that referenced this pull request Oct 9, 2015
[Docs] Add path syntax example
@knowbody knowbody merged commit 9d95102 into remix-run:master Oct 9, 2015
@knowbody
Copy link
Contributor

knowbody commented Oct 9, 2015

thanks

@mjackson
Copy link
Member

mjackson commented Oct 9, 2015

We don't support the ? syntax in 1.0. We use () instead now.
On Fri, Oct 9, 2015 at 1:50 AM Mateusz Zatorski notifications@github.com
wrote:

thanks


Reply to this email directly or view it on GitHub
#2214 (comment).

@knowbody
Copy link
Contributor

knowbody commented Oct 9, 2015

sorry @mjackson, reverting

@fjaguero
Copy link
Contributor Author

Ok, so should we wait it's out of RC to do the same PR but with the correct syntax?

@knowbody
Copy link
Contributor

everything should be correct now @fjaguero

@lock lock bot locked as resolved and limited conversation to collaborators Jan 20, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants