openapi-typescript icon indicating copy to clipboard operation
openapi-typescript copied to clipboard

feat(openAPI-union) Add type support for unions with additionalProperties to provide ts intellisense

Open remy90 opened this issue 6 months ago • 3 comments

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:examples run (only applicable for openapi-typescript)

remy90 avatar Aug 15 '25 13:08 remy90

Deploy request for openapi-ts pending review.

Visit the deploys page to approve it

Name Link
Latest commit 404db56034d9fa96f39d78f6b8e8abc67fa4bcdd

netlify[bot] avatar Aug 15 '25 13:08 netlify[bot]

🦋 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

changeset-bot[bot] avatar Aug 15 '25 14:08 changeset-bot[bot]

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!

remy90 avatar Dec 02 '25 10:12 remy90

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

drwpow avatar Feb 08 '26 23:02 drwpow

Hey @drwpow, Thanks for getting back to me. The merge conflict has been resolved (via rebase) and I've added a changeset.

Thanks again,

remy90 avatar Feb 12 '26 12:02 remy90