Skip to content

Commit

Permalink
Typo in middleware upgrade guide (vercel#40963)
Browse files Browse the repository at this point in the history
<!--
Thanks for opening a PR! Your contribution is much appreciated.
To make sure your PR is handled as smoothly as possible we request that
you follow the checklist sections below.
Choose the right checklist for the change that you're making:
-->

## Bug

- [ ] Related issues linked using `fixes #number`
- [ ] Integration tests added
- [ ] Errors have a helpful link attached, see `contributing.md`

## Feature

- [ ] Implements an existing feature request or RFC. Make sure the
feature request has been accepted for implementation before opening a
PR.
- [ ] Related issues linked using `fixes #number`
- [ ] Integration tests added
- [ ] Documentation added
- [ ] Telemetry added. In case of a feature if it's used or not.
- [ ] Errors have a helpful link attached, see `contributing.md`

## Documentation / Examples

- [ ] Make sure the linting passes by running `pnpm lint`
- [ ] The "examples guidelines" are followed from [our contributing
doc](https://github.com/vercel/next.js/blob/canary/contributing/examples/adding-examples.md)

Co-authored-by: JJ Kasper <jj@jjsweb.site>
  • Loading branch information
2 people authored and BowlingX committed Oct 5, 2022
1 parent 57d582a commit fd6487b
Showing 1 changed file with 8 additions and 8 deletions.
16 changes: 8 additions & 8 deletions errors/middleware-upgrade-guide.md
Expand Up @@ -177,11 +177,11 @@ export default async function handler(req: NextRequest) {

### Summary of changes

| Added | Removed |
| ----------------------- | ------------- |
| `cookie.set` | `cookie` |
| `cookie.delete` | `clearCookie` |
| `cookie.getWithOptions` | `cookies` |
| Added | Removed |
| ------------------------ | ------------- |
| `cookies.set` | `cookie` |
| `cookies.delete` | `clearCookie` |
| `cookies.getWithOptions` | `cookies` |

### Explanation

Expand All @@ -191,9 +191,9 @@ Based on beta feedback, we are changing the Cookies API in `NextRequest` and `Ne

`NextResponse` now has a `cookies` instance with:

- `cookie.delete`
- `cookie.set`
- `cookie.getWithOptions`
- `cookies.delete`
- `cookies.set`
- `cookies.getWithOptions`

As well as other extended methods from `Map`.

Expand Down

0 comments on commit fd6487b

Please sign in to comment.