Export BatchResponseBody interface type

[henhal]

The BatchResponseBody interface is needed as parameter to several public methods, in particular the constructor for BatchResponseContent, but is not exported. This makes it harder to write type-safe code that makes a batch request and passes the body from the response JSON to BatchResponseContent without relying on implicit "any" typing.

A similar PR may already be submitted! Please search among the Pull request before creating one.

Thanks for submitting a pull request! Please provide enough information so that others can review your pull request.

NOTE: PR's will be accepted only in case of appropriate information is provided below

Summary

Export an interface type which is used in other exported interfaces and methods.

Motivation

The BatchResponseBody interface is needed as parameter to several public methods, in particular the constructor for BatchResponseContent, but is not exported. This makes it harder to write type-safe code that makes a batch request and passes the body from the response JSON to BatchResponseContent without relying on implicit "any" typing.

Test plan

Trivial change affecting no code paths except exporting a previously internal type.

Closing issues

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist

• I have read the CONTRIBUTING document.
• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have added tests to cover my changes.
• All new and existing tests passed.

[henhal]

My current workaround is to declare this locally:

type BatchResponseBody = ConstructorParameters<typeof BatchResponseContent>[0];

It would be much cleaner if the type is exported straight from the library.

[nikithauc]

@henhal Thank you for this PR!

Can you please add the export in src/index.ts and src/browser/index.ts?

[ghost]

All CLA requirements met.

[henhal]

@nikithauc Of course, didn't realize members were exported explicitly from index.ts. How come export * from xxx isn't used? Anyway, I added the new type to the two index files. Thanks.

[nikithauc]

@henhal Thank you so much for the update and making this contribution!