-
Notifications
You must be signed in to change notification settings - Fork 5
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
TypeError: Cannot read property 'reduce' of undefined #9
Comments
Are you sure you have the latest version? This was fixed in c19a884 and master version no longer has a line like |
I just pulled the master version following @sbender9's instructions.
Sounds like I need to snag a specific version as it hasn't been merged?
On Mon, Nov 28, 2016 at 9:44 PM Teppo Kurki ***@***.***> wrote:
Are you sure you have the latest version? This was fixed in c19a884
<c19a884>
and master version no longer has a line like unsubscribes =
options.zones.reduce((acc, { per your error message.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#9 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AIWcGLA7RVzWcoU8XYgVycmNY65k1P0wks5rC7vMgaJpZM4K-eoP>
.
--
Steve Mitchell - Editor, SailBits.com <https://www.sailbits.com/> |
[email protected] | 206-659-6911
|
It is merged, see https://github.com/SignalK/signalk-zones/blob/master/index.js#L96. A fresh install of SK server does have that change:
Scott's instructions don't go into detail so I don't know what exactly you did, but I think |
I just did an npm install this evening after updating signalk server (git
pull) and then installing the rest of the other packages he required,
including signal-zones.
I can completely delete my signalk server and start over with a new install
if that would help. Going to take a while to pull down all of the other
modules though.
On Mon, Nov 28, 2016 at 10:33 PM Teppo Kurki ***@***.***> wrote:
It is merged, see
https://github.com/SignalK/signalk-zones/blob/master/index.js#L96.
A fresh install of SK server does have that change:
unsubscribes = (options.zones || []).reduce((acc, {```
Scott's instructions don't go into detail so I don't know what exactly you did, but I think `npm install` in the server's root direction should bring you the update.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#9 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AIWcGB-uQy8aPADo3pV_3FuY4FaC_orTks5rC8dXgaJpZM4K-eoP>
.
--
Steve Mitchell - Editor, SailBits.com <https://www.sailbits.com/> |
[email protected] | 206-659-6911
|
Just delete node_modules/signalk-zones/ and reinstall it with It is really just one file, the one I linked and pointed above, so if all else fails you can install it manually. |
Yeah I already did that several times tonight without success. Already
waxed my signalk directory and re-installing to make sure there are no
other issues.
On Mon, Nov 28, 2016 at 10:42 PM Teppo Kurki ***@***.***> wrote:
Just delete node_modules/signalk-zones/ and reinstall it with npm install
signalk/signalk-zones, that should do the trick.
It is really just one file, the one I linked and pointed above, so if all
else fails you can install it manually.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#9 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AIWcGAI8V_n-ac9mx74gDc1up8ZZ_1Teks5rC8ligaJpZM4K-eoP>
.
--
Steve Mitchell - Editor, SailBits.com <https://www.sailbits.com/> |
[email protected] | 206-659-6911
|
Well that caused a new error after complete reinstall of all components
when I tried to enable push notifications. Off to find that project to
file a bug / investigate!
On Mon, Nov 28, 2016 at 10:57 PM Steve Mitchell ***@***.***> wrote:
Yeah I already did that several times tonight without success. Already
waxed my signalk directory and re-installing to make sure there are no
other issues.
On Mon, Nov 28, 2016 at 10:42 PM Teppo Kurki ***@***.***>
wrote:
Just delete node_modules/signalk-zones/ and reinstall it with npm install
signalk/signalk-zones, that should do the trick.
It is really just one file, the one I linked and pointed above, so if all
else fails you can install it manually.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#9 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AIWcGAI8V_n-ac9mx74gDc1up8ZZ_1Teks5rC8ligaJpZM4K-eoP>
.
--
Steve Mitchell - Editor, SailBits.com <https://www.sailbits.com/> |
***@***.*** | 206-659-6911 <(206)%20659-6911>
--
Steve Mitchell - Editor, SailBits.com <https://www.sailbits.com/> |
[email protected] | 206-659-6911
|
Setting up the various modules for @sbender9 page at https://github.com/sbender9/wilhelmsk-node-server-setup, and after enabling the modules and setting the correct IDs, SignalK exited with the error below. Continuing to try to restart results in the same error.
I'm not sure where these settings are stored so I can zero them out and reproduce...
`/home/pi/signalk-server-node/node_modules/signalk-zones/index.js:81
unsubscribes = options.zones.reduce((acc, {
^
TypeError: Cannot read property 'reduce' of undefined
at Object.plugin.start (/home/pi/signalk-server-node/node_modules/signalk-zones/index.js:81:33)
at /home/pi/signalk-server-node/lib/interfaces/plugins.js:139:16
at FSReqWrap.oncomplete (fs.js:123:15)
pi@grace:~/signalk-server-node$ bin/signalk-server
No settings/defaults.json available
/home/pi/signalk-server-node/node_modules/signalk-zones/index.js:81
unsubscribes = options.zones.reduce((acc, {
^
TypeError: Cannot read property 'reduce' of undefined
at Object.plugin.start (/home/pi/signalk-server-node/node_modules/signalk-zones/index.js:81:33)
at registerPlugin (/home/pi/signalk-server-node/lib/interfaces/plugins.js:115:12)
at fs.readdirSync.filter.forEach.e (/home/pi/signalk-server-node/lib/interfaces/plugins.js:93:7)
at Array.forEach (native)
at startPlugins (/home/pi/signalk-server-node/lib/interfaces/plugins.js:85:68)
at Object.start (/home/pi/signalk-server-node/lib/interfaces/plugins.js:26:7)
at /home/pi/signalk-server-node/lib/index.js:170:30
at /home/pi/signalk-server-node/node_modules/lodash/index.js:3073:15
at Function.forIn (/home/pi/signalk-server-node/node_modules/lodash/index.js:3362:16)
at startInterfaces (/home/pi/signalk-server-node/lib/index.js:162:5)`
The text was updated successfully, but these errors were encountered: