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

feat(zod-openapi): Make context accessible in the doc route #272

Merged
merged 3 commits into from
Nov 28, 2023

Conversation

Karibash
Copy link
Contributor

close: #243

Copy link

changeset-bot bot commented Nov 22, 2023

🦋 Changeset detected

Latest commit: ee703fc

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@hono/zod-openapi Minor

Not sure what this means? Click here to learn what changesets are.

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

@yusukebe
Copy link
Member

Hi @Karibash

I'm sorry to be late.

Looks good! One thing. Could you write the short usage of this feature on README? Thanks!

@Karibash Karibash force-pushed the feature/zod-openapi-doc-context branch from 34e014e to ee703fc Compare November 28, 2023 04:00
@Karibash
Copy link
Contributor Author

@yusukebe
I wasn't sure where to add the explanation, but I added it to the Tips section anyway.
Also, I found some typos, so I fixed them as well.

@yusukebe
Copy link
Member

Hi @Karibash

That place is good!

Let's ship it now. Thanks!

@yusukebe yusukebe merged commit d4aa8ec into honojs:main Nov 28, 2023
@Karibash Karibash deleted the feature/zod-openapi-doc-context branch November 28, 2023 05:41
@github-actions github-actions bot mentioned this pull request Nov 28, 2023
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

Successfully merging this pull request may close these issues.

How access context in app.doc with @hono/zod-openapi ?
2 participants