nextlove
nextlove copied to clipboard
Nextlove Edge Support MVP Steps
In relation to: https://github.com/seamapi/nextlove/pull/82
The PR is too big and dangerous as is. This issue is to design an approach for getting edge-support merged
First let's get some opinions out of the way
- We will introduce
withRouteSpecEdge
to avoid a breaking change. This means it's an EDGE-COMPATIBLE withRouteSpec, it still shares the exact same API and types as withRouteSpec - We will introduce an example app that uses edge e.g.
example-edge-todo-app
, this app also ideally deploys somewhere on cloudflare or vercel edge - Introduce the
edge
term sparingly. Nextlove will work on both edge and lambda, so avoid any "edge-specific" terminology
If you disagree with the above, we can't move on to what's below
- Propose changes to nextlove API e.g. the
req.Response
object that make nextlove compatible with edge. Do not introducewithRouteSpecEdge
or any special edge handling - After the API changes are in, build
withRouteSpecEdge
(NOTE: withRouteSpecEdge is lambda and edge compatible) - Replace
withRouteSpec
withwithRouteSpecEdge
and deprecatewithRouteSpecEdge
CC @itelo @codetheweb @razor-x