Skip to content

Conversation

@bitgopatmcl
Copy link
Contributor

Redefines a "service function" so that the return type is either all
numeric or all keyed. This is a bit of a guess, but the hope is that it
resolves some of the weird typechecking issues we are seeing around
response types.

Redefines a "service function" so that the return type is either all
numeric or all keyed. This is a bit of a guess, but the hope is that it
resolves some of the weird typechecking issues we are seeing around
response types.
@bitgopatmcl bitgopatmcl force-pushed the express-response-type-experiment branch from 63374a2 to 7143029 Compare August 4, 2022 15:26
@bitgopatmcl bitgopatmcl marked this pull request as ready for review August 4, 2022 17:02
@bitgopatmcl bitgopatmcl requested a review from a team as a code owner August 4, 2022 17:02
@bitgopatmcl bitgopatmcl requested a review from jonball4 August 4, 2022 17:02
@jonball4
Copy link
Contributor

jonball4 commented Aug 4, 2022

Works well using solely ResponseType, can't speak for users of the Keyed counter-part.

@ericcrosson-bitgo ericcrosson-bitgo merged commit 82d10af into BitGo:beta Aug 4, 2022
@github-actions
Copy link

github-actions bot commented Aug 4, 2022

🎉 This PR is included in version 0.2.0-beta.8 🎉

The release is available on npm package (@beta dist-tag)

Your semantic-release bot 📦🚀

@github-actions
Copy link

github-actions bot commented Aug 4, 2022

🎉 This PR is included in version 1.0.0-beta.21 🎉

The release is available on npm package (@beta dist-tag)

Your semantic-release bot 📦🚀

@github-actions
Copy link

github-actions bot commented Aug 4, 2022

🎉 This PR is included in version 0.2.0-beta.10 🎉

The release is available on npm package (@beta dist-tag)

Your semantic-release bot 📦🚀

@github-actions
Copy link

github-actions bot commented Aug 4, 2022

🎉 This PR is included in version 0.2.0-beta.14 🎉

The release is available on npm package (@beta dist-tag)

Your semantic-release bot 📦🚀

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants