Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

docs(openapi-fetch): remove line about new examples#2339

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
gzm0 wants to merge1 commit intoopenapi-ts:main
base:main
Choose a base branch
Loading
fromgzm0:no-examples

Conversation

gzm0
Copy link
Contributor

As discussed in the maintainer's chat: It seems that examples mostly showcase how to integrate an (arbitrary) function/client into a specific framework.

As a result, the examples have very little openapi-fetch specific content, but a lot of framework specific content. This, in our opinion, tips the balance of whether the maintenance burden is worth the ease of use.

As discussed in the maintainer's chat: It seems that examples mostlyshowcase how to integrate an (arbitrary) function/client into aspecific framework.As a result, the examples have very little openapi-fetch specificcontent, but a lot of framework specific content. This, in ouropinion, tips the balance of whether the maintenance burden is worththe ease of use.
@gzm0gzm0 requested a review froma team as acode ownerMay 27, 2025 11:14
@gzm0gzm0 requested a review fromdrwpowMay 27, 2025 11:14
@netlifyNetlify
Copy link

netlifybot commentedMay 27, 2025

👷 Deploy request foropenapi-ts pending review.

Visit the deploys page to approve it

NameLink
🔨 Latest commit7a50957

@changeset-botchangeset-bot
Copy link

⚠️ No Changeset found

Latest commit:7a50957

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

@gzm0
Copy link
ContributorAuthor

@HasutoSasaki just to kick off the discussion, we (some of the maintainers,@drwpow and I) feel that adding new examples is not worth the maintenance overhead (reviews, keeping it up to date).

One big reason behind this is that most examples seem to primarily showcase how to call use an arbitrary async in a specific framework. As such, it is mostly framework documentation and not openapi-fetch documentation.

As I'm writing this, I wonder if a middle ground could be to have links to relevant documentation on the framework side.

@HasutoSasaki
Copy link

@gzm0
I’m very sorry for the delay in replying.

First of all, thank you for your thoughtful feedback!

You’re absolutely right — the example I provided was more about demonstrating how to use openapi-typescript with Nuxt 3, rather than showcasing the library itself. I now understand that including it directly in the project may not be ideal, as it would create ongoing maintenance overhead.

That said, I’d like to suggest a possible alternative: what if I host the Nuxt 3 + openapi-typescript example in my own repository, and you link to it from this project? This way, it could still help users without adding any maintenance burden to this repository.

@gzm0
Copy link
ContributorAuthor

gzm0 commentedJun 2, 2025

@HasutoSasaki I think that is an excellent idea! I think managing the links to such examples is absolutely within reason for the core maintainers to do.

@HasutoSasaki
Copy link

@gzm0
Thank you!
I’ll prepare the example in my own repository, so please give me a little more time.

gzm0 reacted with thumbs up emoji

@HasutoSasaki
Copy link

@gzm0
Sorry for the delay
I have a sample in my repository
Would you like to set up a new PR to add the link?

nuxt3 example repository

@gzm0
Copy link
ContributorAuthor

I have a sample in my repository

Fantastic!

Would you like to set up a new PR to add the link?

I would suggest you just amend your existing PR (#2331) and the I'll close this one (or amend it to clarify examples should be externally hosted).

HasutoSasaki reacted with thumbs up emoji

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@drwpowdrwpowAwaiting requested review from drwpowdrwpow is a code owner automatically assigned from openapi-ts/maintainers

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@gzm0@HasutoSasaki

[8]ページ先頭

©2009-2025 Movatter.jp