Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Edge Browser Compatible #10

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Edge Browser Compatible #10

wants to merge 4 commits into from

Commits on May 27, 2018

  1. TextEncoder/Decoder polyfill

    @arasouli91
    arasouli91 authored May 27, 2018
    Configuration menu
    Copy the full SHA
    50b52e1 View commit details
    Browse the repository at this point in the history

  2. Replaced spread syntax with Object.assign() 

    Not sure why this was breaking compatibility with Edge, spread syntax is supposed to be supported by Edge.
    @arasouli91
    arasouli91 authored May 27, 2018
    Configuration menu
    Copy the full SHA
    d1ac08e View commit details
    Browse the repository at this point in the history

  3. Added short-circuit condition to catch undefined crypto 

    I believe the issue is crypto is a member of Window, so it is not accessible in a worker.
    @arasouli91
    arasouli91 authored May 27, 2018
    Configuration menu
    Copy the full SHA
    5f973d3 View commit details
    Browse the repository at this point in the history

  4. Import encoding polyfill only if undefined 

    Polyfill for Edge.
    @arasouli91
    arasouli91 authored May 27, 2018
    Configuration menu
    Copy the full SHA
    0970b51 View commit details
    Browse the repository at this point in the history