hey-api / openapi-ts

🚀 The OpenAPI to TypeScript codegen. Generate clients, SDKs, validators, and more. Support: @mrlubos
https://heyapi.dev
Other
1.39k stars 107 forks source link

feat: Add draft angular-query-experimental support #1218

Closed max-scopp closed 2 weeks ago

max-scopp commented 3 weeks ago

Closes #1072 This PR introduces preliminary support for Angular Query Experimental, functioning similarly to the existing React Query plugin.

Reasoning

We’re exploring ways to integrate @hey-api at work, but legacy Angular support is a significant barrier. This PR aims to offer an initial foundation for community collaboration and feedback. While this topic isn’t on a fast-tracked timeline internally, the basic implementation here is meant to initiate development in this direction.

Next Steps

I'm happy to extend examples and provide continued support for this PR, with the ultimate goal of making it merge-ready. The hope is that this initial setup can evolve with community input, building Angular support that meets the needs of various projects.

stackblitz[bot] commented 3 weeks ago

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

vercel[bot] commented 3 weeks ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
hey-api-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 1, 2024 9:03am
changeset-bot[bot] commented 3 weeks ago

⚠️ No Changeset found

Latest commit: f40c8c83e5942aab8074da17d8685f22cb5db59c

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

max-scopp commented 3 weeks ago

I just got myself a glass of rum to celebrate! I haven't found a discord server link, but you can find me under "maxscopp" on Discord and we can discuss the rest.

Thanks for the insanely fast response!

codecov[bot] commented 3 weeks ago

Codecov Report

Attention: Patch coverage is 89.13043% with 5 lines in your changes missing coverage. Please review.

Project coverage is 73.19%. Comparing base (13e0065) to head (f40c8c8). Report is 10 commits behind head on main.

Files with missing lines Patch % Lines
...ins/@tanstack/angular-query-experimental/config.ts 86.95% 3 Missing :warning:
.../src/plugins/@tanstack/query-core/plugin-legacy.ts 85.71% 1 Missing :warning:
...napi-ts/src/plugins/@tanstack/query-core/plugin.ts 85.71% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1218 +/- ## ========================================== + Coverage 73.15% 73.19% +0.03% ========================================== Files 125 127 +2 Lines 17662 17706 +44 Branches 1732 1732 ========================================== + Hits 12921 12960 +39 - Misses 4738 4743 +5 Partials 3 3 ``` | [Flag](https://app.codecov.io/gh/hey-api/openapi-ts/pull/1218/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=hey-api) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/hey-api/openapi-ts/pull/1218/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=hey-api) | `73.19% <89.13%> (+0.03%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=hey-api#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.