Skip to content

docs: fix code error in middleware-auth.md #2460

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

docs: fix code error in middleware-auth.md #2460

wants to merge 1 commit into from
+1 −3

Conversation

adamwdennis
Copy link

Changes

Just an issue with the middleware.md code.

How to Review

Look at the fix.

Checklist

  • Unit tests updated
  • docs/ updated (if necessary)
  • pnpm run update:examples run (only applicable for openapi-typescript)

@adamwdennis adamwdennis requested a review from a team as a code owner September 23, 2025 14:08
@adamwdennis adamwdennis requested a review from drwpow September 23, 2025 14:08
@changeset-bot changeset-bot
Copy link

changeset-bot bot commented Sep 23, 2025

⚠️ No Changeset found

Latest commit: 440b7e7

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@netlify Netlify
Copy link

netlify bot commented Sep 23, 2025 •
edited
Loading

❌ Deploy Preview for openapi-ts failed.

Name Link
🔨 Latest commit 440b7e7
🔍 Latest deploy log https://app.netlify.com/projects/openapi-ts/deploys/68d2a9ca9510e4000890a17c

@adamwdennis adamwdennis changed the title docs: fix code error middleware-auth.md docs: fix code error in middleware-auth.md Sep 23, 2025
@adamwdennis adamwdennis changed the title docs: fix code error in middleware-auth.md docs: fix code error in middleware-auth.md Sep 23, 2025
gzm0
gzm0 approved these changes Oct 2, 2025
View reviewed changes
Copy link
Contributor

@gzm0 gzm0 left a comment

Choose a reason for hiding this comment

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

TY!

@gzm0
Copy link
Contributor

gzm0 commented Oct 2, 2025

@drwpow do you think you could have a look at netlify here?

@gzm0 gzm0 added openapi-ts Relevant to the openapi-typescript library openapi-fetch Relevant to the openapi-fetch library and removed openapi-ts Relevant to the openapi-typescript library labels Oct 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gzm0 gzm0 gzm0 approved these changes

@drwpow drwpow Awaiting requested review from drwpow drwpow is a code owner automatically assigned from openapi-ts/maintainers

Assignees
No one assigned
Labels
openapi-fetch Relevant to the openapi-fetch library
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@adamwdennis @gzm0