Skip to content
This repository was archived by the owner on Jan 25, 2025. It is now read-only.

fix: add support for Web Worker#270

Merged
marcandrews merged 1 commit intotwitch-js:nextfrom
ssut:fix/web-workers
Dec 27, 2020
Merged

fix: add support for Web Worker#270
marcandrews merged 1 commit intotwitch-js:nextfrom
ssut:fix/web-workers

Conversation

@ssut
Copy link
Contributor

@ssut ssut commented Dec 24, 2020

"window" prevents fetch and FormData to work in a service and web worker context.

@codecov
Copy link

codecov bot commented Dec 24, 2020

Codecov Report

Merging #270 (2486e6c) into next (f4b3203) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@         Coverage Diff         @@
##           next   #270   +/-   ##
===================================
  Coverage    98%    98%           
===================================
  Files        25     25           
  Lines      1277   1277           
  Branches    246    247    +1     
===================================
  Hits       1250   1250           
  Misses       26     26           
  Partials      1      1           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f4b3203...2486e6c. Read the comment docs.

Copy link
Member

@marcandrews marcandrews left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great 🐬

Please rename the commit and PR title to Bugfix: add support for Web Worker.

@marcandrews marcandrews merged commit 4684cbd into twitch-js:next Dec 27, 2020
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.

2 participants