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

Update middleware matcher docs to mention has matching #63533

Merged
merged 2 commits into from
Mar 20, 2024

Conversation

ijjk
Copy link
Member

@ijjk ijjk commented Mar 20, 2024

Looks like we were only documenting the missing matcher use case although has is also supported and we weren't mentioning they can combined as well.

x-ref: slack thread

Closes NEXT-2883

@ijjk ijjk requested review from a team as code owners March 20, 2024 20:44
@ijjk ijjk requested review from jh3y and delbaoliveira and removed request for a team March 20, 2024 20:44
Co-authored-by: Zack Tanner <zacktanner@gmail.com>
@ijjk ijjk enabled auto-merge (squash) March 20, 2024 20:51
@ijjk ijjk merged commit 00bcd55 into vercel:canary Mar 20, 2024
29 checks passed
@ijjk ijjk deleted the docs/has-matcher-support branch March 20, 2024 21:08
@github-actions github-actions bot added the locked label Apr 4, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
created-by: Next.js team PRs by the Next.js team. locked
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants