Resolves #89 and partially resolves #97

adlnet / xAPIWrapper

Wrapper to simplify communication to an LRS

https://adlnet.gov/projects/xapi/

Apache License 2.0

219 stars 114 forks source link

Closed brian-learningpool closed 5 years ago

brian-learningpool commented 6 years ago

Note that #97 still requires some kind of shim for the URL object in IE, but this commit contains two fixes for IE 11, even with the shim in place.

The 'withCredentials' property could be set to undefined and caused the code to error, but a typeof check corrects this (thanks @shigil)
The getbase() errored with xAPI launch because 'protocol' and 'port' were undefined

brian-learningpool commented 5 years ago

@vbhayden, this PR has been hanging around for some time, might it be possible to get this merged?