Feature/pages router

[jsdalton]

Purpose

• Provide an API route for pages router to enable Draft Mode
• Clean up and consolidate shared code

Description

TODO

• Expose pages-router package
• Add tests for utils

Testing steps

Breaking Changes

Design, Documentation, and/or References

Deployment

[matthew-gordon]

Looks great. I think we just need to add to the exports and also add pages-router to the files in the package.json to make sure its referenced correctly:

...
  "files": [
    "dist",
    "app-router",
    "pages-router"
  ],
  "exports": {
    ".": {
      "import": "./dist/index.js",
      "require": "./dist/index.js",
      "types": "./dist/index.d.ts"
    },
    "./app-router": {
      "import": "./dist/app-router/handlers/enable-draftjs",
      "require": "./dist/app-router/handlers/enable-draft.js",
      "types": "./dist/app-router/handlers/enable-draft.d.ts"
    },
    "./pages-router": {
      "import": "./dist/pages-router/handlers/enable-draft.js",
      "require": "./dist/pages-router/handlers/enable-draft.js",
      "types": "./dist/pages-router/handlers/enable-draft.d.ts"
    }
  },
...

[matthew-gordon]

ok tested this locally, just a few changes needed to get this working as expected, the new pattern for this import/export will look like this in a project:

export { default as handler } from "@contentful/vercel-nextjs-toolkit/pages-router";

Pages router errors out if we dont have a default export handler from the api file.

[matthew-gordon]

This looks good pending the few things we talked about shaping this back up last week 👍