Skip to content
This repository has been archived by the owner on Mar 12, 2020. It is now read-only.

support TextDecoder in older nodejs env #119

Merged
merged 2 commits into from
Jun 20, 2019

Conversation

connectdotz
Copy link
Contributor

@connectdotz connectdotz commented May 25, 2019

fixing #118

  • added a simple check for global TextDecoder, if not available, fallback to node 'util' package.
  • added test for helper

tested it with both node 10.15.1 and 12.3.1

p.s. this is my first contribution to your wonderful work, please let me know if I missed anything.

Signed-off-by: vsun <vsun@connectdotz.com>

Date:      Sat May 25 16:15:31 2019 -0400

On branch TextDecoder-compatibility
Your branch is up-to-date with 'origin/TextDecoder-compatibility'.

Changes to be committed:
    new file:   src/helpers/__test__/handlers.test.ts
    modified:   src/helpers/handlers.ts

fixing textileio#118

added a simple check for global TextDecoder, if not available, fallback to node 'util' package.
@carsonfarmer carsonfarmer merged commit 4139f4b into textileio:master Jun 20, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants