stevenflesch / ring-to-android-tv

Node application to send Ring API events/pictures to Android TV via PiPup.
MIT License
33 stars 10 forks source link

TypeError: Cannot read properties of undefined (reading 'subscribe') #32

Open ZuberousMaximus opened 1 year ago

ZuberousMaximus commented 1 year ago

I know this isn't mantained anymore but thought I'd add an issue I'm facing and if someone knows how I can get around this?

Amazon Linux 2023 [EC2] Node.js v18.16.0

Steps to reproduce:

  1. Install based on instructions on README
  2. Run: node app.js
  3. Unable to start and following error code is presented:

Error Code:

Refresh Token Updated
Found location: [LOCATION]
     - Found Doorbell Gen 2 named Front Door.
/home/ec2-user/ring-to-android-tv/app.js:142
            camera.onNewDing.subscribe(async ding => {
                             ^

TypeError: Cannot read properties of undefined (reading 'subscribe')
    at /home/ec2-user/ring-to-android-tv/app.js:142:30
    at Array.forEach (<anonymous>)
    at /home/ec2-user/ring-to-android-tv/app.js:138:26
    at Array.forEach (<anonymous>)
    at startCameraPolling (/home/ec2-user/ring-to-android-tv/app.js:134:15)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
durag5 commented 1 year ago

Not a direct answer but I actually moved to home assistant which provides the same functionality with more community support.

On Tue, May 23, 2023, 6:58 AM ZuberousMaximus @.***> wrote:

I know this isn't mantained anymore but thought I'd add an issue I'm facing and if someone knows how I can get around this?

Amazon Linux 2023 [EC2] Node.js v18.16.0

Steps to reproduce:

  1. Install based on instructions on README
  2. Run: node app.js
  3. Unable to start and following error code is presented:

Error Code:

Refresh Token Updated Found location: [LOCATION]

  • Found Doorbell Gen 2 named Front Door. /home/ec2-user/ring-to-android-tv/app.js:142 camera.onNewDing.subscribe(async ding => { ^

TypeError: Cannot read properties of undefined (reading 'subscribe') at /home/ec2-user/ring-to-android-tv/app.js:142:30 at Array.forEach () at /home/ec2-user/ring-to-android-tv/app.js:138:26 at Array.forEach () at startCameraPolling (/home/ec2-user/ring-to-android-tv/app.js:134:15) at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

— Reply to this email directly, view it on GitHub https://github.com/stevenflesch/ring-to-android-tv/issues/32, or unsubscribe https://github.com/notifications/unsubscribe-auth/ARKBETMABXIORRGAEKHHGXLXHS7AXANCNFSM6AAAAAAYL6EQVI . You are receiving this because you are subscribed to this thread.Message ID: @.***>

stevenflesch commented 1 year ago

Are you able to successfully use the --test flag with the script?

stevenflesch commented 1 year ago

Not a direct answer but I actually moved to home assistant which provides the same functionality with more community support. On Tue, May 23, 2023, 6:58 AM ZuberousMaximus @.> wrote: I know this isn't mantained anymore but thought I'd add an issue I'm facing and if someone knows how I can get around this? Amazon Linux 2023 [EC2] Node.js v18.16.0 Steps to reproduce: 1. Install based on instructions on README 2. Run: node app.js 3. Unable to start and following error code is presented: Error Code: Refresh Token Updated Found location: [LOCATION] - Found Doorbell Gen 2 named Front Door. /home/ec2-user/ring-to-android-tv/app.js:142 camera.onNewDing.subscribe(async ding => { ^ TypeError: Cannot read properties of undefined (reading 'subscribe') at /home/ec2-user/ring-to-android-tv/app.js:142:30 at Array.forEach () at /home/ec2-user/ring-to-android-tv/app.js:138:26 at Array.forEach () at startCameraPolling (/home/ec2-user/ring-to-android-tv/app.js:134:15) at process.processTicksAndRejections (node:internal/process/task_queues:95:5) — Reply to this email directly, view it on GitHub <#32>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ARKBETMABXIORRGAEKHHGXLXHS7AXANCNFSM6AAAAAAYL6EQVI . You are receiving this because you are subscribed to this thread.Message ID: @.>

Interesting, I use home assistant as well. What are you using to display notifications on Android TV?

ZuberousMaximus commented 1 year ago

Are you able to successfully use the --test flag with the script?

yes weirdly works fine with the test flag

$ node app.js --test

Attempting to get demo snapshot...
Refresh Token Updated
Attempting to get snapshot for location #0, camera #0
Snapshot size: 30 kb
Snapshot saved!
Sent notification successfully: Test Snapshot - This is a test snapshot message!