Skip to content
This repository has been archived by the owner on May 10, 2021. It is now read-only.

Catch-all routes (reopening of PR#1) #5

Merged
merged 7 commits into from
Apr 19, 2020
Merged

Catch-all routes (reopening of PR#1) #5

merged 7 commits into from
Apr 19, 2020

Conversation

FinnWoelm
Copy link
Collaborator

This is re-opening the pull request proposed by @spencewood. I accidentally messed up their pull request #1 🙈 I'll keep my fingers off force pushing from now on...

@spencewood:
This simply pulls in the latest code from https://github.com/danielcondemarin/serverless-next.js/blob/master/packages/serverless-nextjs-component/lib/expressifyDynamicRoute.js which supports dynamic catch all routes.

spencewood and others added 6 commits March 7, 2020 15:11
A new version of serverless-nextjs-component has been released. It
introduces support for dynamic catch-all routes. Integrating these
changes means that we can also support dynamic catch-all routes.
Create an adaptation of serverless-next.js' expressifyDynamicRoute
function: getNetlifyRoute. Our getNetlifyRoute function differs from
the expressifyDynamicRoute in that it converts catch-all routes from
[...params] to /:* rather than to /:params*. The latter does not work
with Netlify routing, but our version does.
@FinnWoelm FinnWoelm mentioned this pull request Apr 19, 2020
Acknowledge their help with adding support for catch-all routes.
@FinnWoelm FinnWoelm merged commit 4aad3df into master Apr 19, 2020
@FinnWoelm FinnWoelm deleted the catch-all-routes branch April 19, 2020 14:25
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.

2 participants