nabbl / node-red-contrib-google-home-notify

Node-Red Node to send Text messages for Google Home to read from
MIT License
17 stars 10 forks source link

0.0.4 not working. Error #4

Closed korniza closed 6 years ago

korniza commented 7 years ago

I get an error when i simple try to test google notify

"SyntaxError: Block-scoped declarations (let, const, function, class) not yet supported outside strict mode" I have placed the IP of the google Home device but it reports under the box on node-red "please creeate & select a config node:. As I am new to node-red, it may be my fault. Is something I missing (dependencies for example)? please help!

nabbl commented 7 years ago

Do you have more informations about what Node-Version you are using?

korniza commented 7 years ago

24 Oct 21:32:15 - [info] Node-RED version: v0.17.5 24 Oct 21:32:15 - [info] Node.js version: v4.8.4 24 Oct 21:32:15 - [info] Linux 4.4.0-97-generic x64 LE 24 Oct 21:32:15 - [info] Loading palette nodes 24 Oct 21:32:16 - [info] Dashboard version 2.5.1 started at /ui

jenskastensson commented 7 years ago

I'm running NodeJs 4 too I've modified node_modules\google-home-notify\google-home-notifier.js Add line 0 'use strict';

Change line 32 to Googletts(text, language, 1).then(function (url) {

On Tue, Oct 24, 2017 at 8:32 PM, Zac K. notifications@github.com wrote:

24 Oct 21:32:15 - [info] Node-RED version: v0.17.5 24 Oct 21:32:15 - [info] Node.js version: v4.8.4 24 Oct 21:32:15 - [info] Linux 4.4.0-97-generic x64 LE 24 Oct 21:32:15 - [info] Loading palette nodes 24 Oct 21:32:16 - [info] Dashboard version 2.5.1 started at /ui

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/nabbl/node-red-contrib-google-home-notify/issues/4#issuecomment-339088062, or mute the thread https://github.com/notifications/unsubscribe-auth/AGTzwQVzZfKvMBa0sMeqCS_JDPzUuGDKks5svi1JgaJpZM4QEurC .

nabbl commented 7 years ago

Yes. When you are using a Node-version smaller than v7 you need to set "use strict".

Maybe I should change the required node version in the package.json or set use strict in the js file.

jenskastensson commented 7 years ago

Personally I would appreciate native support of Node4!

On Wed, Oct 25, 2017 at 9:09 AM, Daniel Kappler notifications@github.com wrote:

Yes. When you are using a Node-version smaller than v7 you need to set "use strict".

Maybe I should change the required node version in the package.json or set use strict in the js file.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/nabbl/node-red-contrib-google-home-notify/issues/4#issuecomment-339235767, or mute the thread https://github.com/notifications/unsubscribe-auth/AGTzwQjOScv_24gbawUZQPn39onLptfGks5svt6PgaJpZM4QEurC .

korniza commented 7 years ago

it worked! thanks for help! it would be great to check node version

nabbl commented 6 years ago

Should be fixed in the newest version (0.0.5)