Skip to content

docs: Better documentation on File Transfer errors.#56

Merged
OS-pedrogustavobilro merged 8 commits into
mainfrom
docs/RMET-4893
Jan 30, 2026
Merged

docs: Better documentation on File Transfer errors.#56
OS-pedrogustavobilro merged 8 commits into
mainfrom
docs/RMET-4893

Conversation

@OS-pedrogustavobilro

@OS-pedrogustavobilro OS-pedrogustavobilro commented Jan 26, 2026

Copy link
Copy Markdown
Contributor

Partially addresses #18 - Another PR will be open to address response body not arriving in iOS.

See commit messages for a list of changes.

Note that while this PR adds headers to FileTransferError interface, this is not a new feature, but rather a documentation update as this field was already being returned by native before, it was just not made explicit on TypeScript API.

This is not a new parameter, it was already being returned by the plugin, just not available in the interface
FileTransferError was not being used explicitly, hence why we added an internal field to the API so that docgen picks it up.
@OS-pedrogustavobilro OS-pedrogustavobilro requested a review from a team January 26, 2026 17:35
@OS-pedrogustavobilro OS-pedrogustavobilro merged commit 49e56aa into main Jan 30, 2026
8 checks passed
@OS-pedrogustavobilro OS-pedrogustavobilro deleted the docs/RMET-4893 branch January 30, 2026 09:34
OS-pedrogustavobilro added a commit that referenced this pull request Feb 11, 2026
* docs: Make explict regarding http errors

* docs: FileTransferError Android / iOS only

* docs: mention headers

This is not a new parameter, it was already being returned by the plugin, just not available in the interface

* docs: Add error handling to examples

* chore: Make FileTransferError discoverable by docgen

FileTransferError was not being used explicitly, hence why we added an internal field to the API so that docgen picks it up.

* chore: fix lint issues

* chore: fix lint issues in web.ts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants