Running 1.12.0 in an electron environment about ads-client HOT 11 CLOSED

Hey, sry been quite busy with some other stuff. I hope to give you feedback within this week. However, commit looks good 👍

from ads-client.

Hi Andy!

There seems no reason why not to update it. However, just wondering, shouldn't the "iconv-lite": "^0.5.2" in package.json mean that it should install the most recent 0.x.x version always? So 0.6.2 would be installed? Or am I understanding something wrong?

Ps. Been wanting to try Electron and this library too, but haven't got time yet. Let me know how it goes.

from ads-client.

Hey, thanks for the quick reply.

That holds true for all Versions greater than 1.0.0. However, npm treats the 0.x.y version as pre-release and does not apply the same logic (as it is expected to have breaking changes between 0.5.y and 0.6.y for example).

from ads-client.

Oh I see. Thanks for the info!

I'll update the package.json in few days.

from ads-client.

Thanks alot!

I also had issues with following statements (ads-client-ads.js)
return ((ret = Object.keys(this).find(key => this[key] == "asd")) != null ? ret : 'UNKNOWN')
due to the fact that ret is not declared as let/const/var. While this works in plain JS, in my environment it did not.
I replaced it with following code to get it running:
return (Object.keys(this).find(key => this[key] == value)) || 'UNKNOWN';

Should I open another issue for this?

Best regards
Andy

from ads-client.

No need to, I'll check them out too! Thanks for reporting, The original code makes no sense :D

from ads-client.

Sorry for delay, been a little busy. I'll do this today or tomorrow!

from ads-client.

No worries, just text when you're done and I'll verify the new version. Thanks for your effort!

from ads-client.

Just released new version, let me know how it goes!

from ads-client.

Hey @enenkel! Any news? :)

from ads-client.

I'm closing this now. If there are problems you can create a new issue.

from ads-client.