feat(openAPI-union) Add type support for unions with additionalProperties to provide ts intellisense
Thank you for your work on this package, much appreciated! I came across string & {} recently to get 'loose autocomplete' on a string and thought it might help here.
Changes
What does this PR change? Link to any related issue(s). A union of type "A" | "B" | "C" will now provide intellisense options from the union but also accept non-union string entries
How to Review
How can a reviewer review your changes? What should be kept in mind for this review?
Checklist
- [x] Unit tests updated
- [x]
docs/updated (if necessary) - [x]
pnpm run update:examplesrun (only applicable for openapi-typescript)
Deploy request for openapi-ts pending review.
Visit the deploys page to approve it
| Name | Link |
|---|---|
| Latest commit | 404db56034d9fa96f39d78f6b8e8abc67fa4bcdd |
🦋 Changeset detected
Latest commit: 404db56034d9fa96f39d78f6b8e8abc67fa4bcdd
The changes in this PR will be included in the next version bump.
This PR includes changesets to release 1 package
| Name | Type |
|---|---|
| openapi-typescript | Patch |
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
Hey @drwpow @gzm0 Any chance of a review on this? openapi has served me well on a recent project and I wanted to give back in a small way. LMK if you have any thoughts!
Thanks for submitting this! Sorry for the delay in reviewing. There’s a slight conflict with another PR that recently got merged. If you’re able to resolve the conflict, and tests are still passing, would be happy to merge, thank you!
Also please be sure to add a changeset (see comment) so this can be versioned & released
Hey @drwpow, Thanks for getting back to me. The merge conflict has been resolved (via rebase) and I've added a changeset.
Thanks again,