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

docs: Redirecting fix route handler path #72617

Merged
merged 3 commits into from
Nov 21, 2024

Conversation

hyungjikim
Copy link
Contributor

@hyungjikim hyungjikim commented Nov 12, 2024

Hi, Team.
The path of route handler seems to be missing api and I'd like to fix it.
(middleware.ts above the route handler file, it requests to /api/redirects..)
image

Thank you :)

Improving Documentation

  • correct route handler path

@ijjk ijjk added the Documentation Related to Next.js' official documentation. label Nov 12, 2024
@ijjk
Copy link
Member

ijjk commented Nov 12, 2024

Allow CI Workflow Run

  • approve CI run for commit: eb4c91a

Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer

Sorry, something went wrong.

@hyungjikim hyungjikim changed the title fix: route handler path docs: Redirecting fix route handler path Nov 16, 2024

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Copy link
Contributor

@delbaoliveira delbaoliveira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@delbaoliveira delbaoliveira merged commit 4875c58 into vercel:canary Nov 21, 2024
39 checks passed
samcx added a commit that referenced this pull request Nov 22, 2024

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
## Description
Follow up #72617.
I have already fixed at #72944.
`app/api/redirects/route.ts` is ts extension.

### Improving Documentation

- [x] Run `pnpm prettier-fix` to fix formatting issues before opening
the PR.
- [x] Read the Docs Contribution Guide to ensure your contribution
follows the docs guidelines:
https://nextjs.org/docs/community/contribution-guide

Co-authored-by: Sam Ko <sam@vercel.com>
wyattjoh pushed a commit that referenced this pull request Nov 28, 2024

Unverified

This user has not yet uploaded their public signing key.
Hi, Team.
The path of route handler seems to be missing `api` and I'd like to fix
it.
(`middleware.ts` above the route handler file, it requests to
`/api/redirects..`)
<img width="650" alt="image"
src="https://github.com/user-attachments/assets/f9ff3034-de36-42ec-8d85-a4a923005fdf">

Thank you :) 

### Improving Documentation

- correct route handler path

---------

Co-authored-by: hyungji <hj@doodlin.co.kr>
Co-authored-by: Delba de Oliveira <32464864+delbaoliveira@users.noreply.github.com>
wyattjoh pushed a commit that referenced this pull request Nov 28, 2024

Unverified

This user has not yet uploaded their public signing key.
## Description
Follow up #72617.
I have already fixed at #72944.
`app/api/redirects/route.ts` is ts extension.

### Improving Documentation

- [x] Run `pnpm prettier-fix` to fix formatting issues before opening
the PR.
- [x] Read the Docs Contribution Guide to ensure your contribution
follows the docs guidelines:
https://nextjs.org/docs/community/contribution-guide

Co-authored-by: Sam Ko <sam@vercel.com>
@github-actions github-actions bot added the locked label Dec 6, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 6, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Documentation Related to Next.js' official documentation. locked
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants