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.

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

Fix "Invalid calling object" in Edge and ES6 syntax error in IE #321

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

AlexeyMz
Copy link

Included changes:

  • Wrap indirect setImmediate into function to prevent "Invalid calling object" error in IE/Edge (due to a bug it's an error to call setImmediate indirectly).
  • Fix babel-loader include condition to transpile rdf-canonize dependency.

This PR fixes issue #319

* Wrap indirect `setImmediate` into function to prevent "Invalid calling object" error in IE/Edge (due to a bug it's an error to call `setImmediate` indirectly).
* Fix `babel-loader` include condition to transpile `rdf-canonize` dependency.
@AlexeyMz
Copy link
Author

AlexeyMz commented Sep 2, 2019

Hi @davidlehn @dlongley
I rebased PR on latest develop to pass the tests. Is there anything else preventing the PR to be merged?

@andreikun
Copy link

andreikun commented Sep 13, 2019

We encountered same issue and we end up importing from dist instead of primary entrypoint something like this:
const jsonld = require('jsonld/dist/jsonld.js');

Unfortunately, we still experience "Unhandled promise rejection TypeError: Invalid calling object" and this fix would really help us as we do not want to start using babel loader for 3rd party libraries and node modules.

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.

None yet

2 participants