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

Create an api route manifest #744

Open
conico974 opened this issue Feb 17, 2025 · 1 comment
Open

Create an api route manifest #744

conico974 opened this issue Feb 17, 2025 · 1 comment

Comments

@conico974
Copy link
Contributor

For this case we need to create an api-route manifest to properly check

// Api routes are not present in the routes manifest except if they're not behind /api
// /api even if it's a page route doesn't get generated in the manifest
// Ideally we would need to properly check api routes here
const isApiRoute =
internalEvent.rawPath === apiPrefix ||
internalEvent.rawPath.startsWith(`${apiPrefix}/`);

@vicb
Copy link
Contributor

vicb commented Feb 17, 2025

Thanks for the issue.

It would be nice to add an e2e for API routes in the pages-router examples as part of this effort.

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

No branches or pull requests

2 participants