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

Object.assign() on not-so-old nwjs #111

Closed
vankasteelj opened this issue Mar 26, 2016 · 1 comment
Closed

Object.assign() on not-so-old nwjs #111

vankasteelj opened this issue Mar 26, 2016 · 1 comment
Labels

Comments

@vankasteelj
Copy link

Hi,

Your new versions (I use 4.x normally) aren't compatible with nwjs 0.12.3, because of the Object.assign() in "create" method.

May I suggest you to use a polyfill (plain or as a dep) to allow support of a little older platforms?

I'll PR 2 different possibilities to solve the matter, if you like one of them and are OK with supporting a greater range of platforms, please merge it :)

Thanks!

@oozcitak
Copy link
Owner

This should do it: 9972cff

Thank you

tlancina added a commit to ionic-team/ionic-app-lib that referenced this issue Mar 30, 2016
Verison of xml2js we were on had a very broad semver range for its
dependency on xmlbuilder, which has had several breaking changes.
Update to narrow that range. Related:
oozcitak/xmlbuilder-js#111.
tlancina added a commit to ionic-team/ionic-app-lib that referenced this issue Mar 31, 2016
Verison of xml2js we were on had a very broad semver range for its
dependency on xmlbuilder, which has had several breaking changes.
Update to narrow that range. Related:
oozcitak/xmlbuilder-js#111.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants