Hello, your application is excellent, and you’ve done a great job. However, like others on version 2023, I’m encountering the error “Cannot read properties of undefined (reading ‘name’).” Everything works fine; however, if a card appears after the Pushover card, it won’t be executed because it’s in an error state. I don’t know anything about programming, and I have the impression that, for now, no one wants to take over the code. Please, can you fix the problem? The only help I can offer is this: I asked ChatGPT if it had any ideas, and while it might say anything, it might help you. Here’s what it says:
(Note: The translation includes both the original text and the requested translation into English for context. The provided English translation addresses the technical issue mentioned in the original text.)
"The error message ‘Cannot read properties of undefined (reading ‘name’)’ means that the code is trying to access the ‘name’ property of an object that is undefined (null or not defined).
To fix this issue, you need to make sure that the object you’re trying to access the ‘name’ property on is defined and contains that property. In the code you provided, the line that generates this error is the following: let pDevice = args.device.name;. It seems that ‘args.device’ can be undefined in some situations. You can add a check to ensure that ‘args.device’ is defined before trying to access its ‘name’ property. For example: let pDevice = args.device ? args.device.name : ‘Fallback Name’;. This will prevent the error if ‘args.device’ is undefined. You can replace ‘Fallback Name’ with an appropriate default value if needed. Make sure to make this modification in all similar occurrences in your code to avoid other similar errors. I hope this helps and allows you to resolve the final issue for the transition to Homey 2023. Thank you."
My intentions are good. I am new to Homey and was looking for a notification with snapshot support (images from camera). After google-search, Pushover was the answer.
Found out missing support for Homey Pro 2023 - the app was outdated/unsupported.
I created a new app and went for app store submission.
THEN found out an updated Pushover Notification app was in the making and in TEST app-store (dont remember how I found it). I dont know how long this app/version has been stalled in TEST. Also I was expericening issues with undefined and dont have the need for “user/group” specification/support.
I’ve then realised the features of the app was kinda of overloaded (user, focus, delay etc.) = wanted some basic: title + message. More importantly the lack of support for snapshot made me want to proceed with this app submission.
I’ve just PM’ed @PandaNL about this. Did not know how to contact this fellow developer before.
Do not want to push someones buttons or take over someone else’s terroritory. Best intentions…
At this moment i use both apps and in the new one it isn’t possible to choose sound or priority.
It would be great if the 2 apps integrate to 1 app with all that together!
Hi all, I will submit a new version with priority and sound tonight. App submission will not take long, typically one day or so. (if you cannot wait: Pushover App for Homey | Homey)
I can post an update here, when new version is out.
I’ve done some refactoring. Please help me test
I need feedback with the following:
High Priority: with high priority both expire and retry is required.
Retry: Pushover will resend same notification with up to 30-seconds of delay
Expire: for how long the notification will continue to be retried for delivery
DO YOU GUYS WANT THIS? I mean it’s kinda complex with two intervals - we could setup 2-3 options.
If you want it, we need overloads for both notification-types: with image/snapshot and a version without image/snapshot. Future version
Btw. PandaNL have reached out. I will carry the future development for now.
Lol, what a coincidence … as I just polished some things up and submitted my version to athom.
Mine is different in the fact that you can add keys/variables ( via labels ) in the cards and thus can send from different apps and/or send to more than 1 user.
It’s something i really wanted and needed and couldnt find.
Let’s hope we can make everyone happy by having multiple apps
Unfortunately, your app has not passed the certification process, due to the following:
— Your app is similar to an already newly published app namely Pushover by Kim Kokholm. As you might know we have a one app per brand policy. This makes it clear and easy for the end-users.
Well, there’s that … i guess you can’t use my version then.
Godspeed @Kim_Kokholm
I believe that this is working as well. You cannot select the group, but when I send to device (must create one) with the group name then it works for me.
So I have created new device and the devicename is then the name of the group. Maybe it works for you as well.
I’ve just pushed a new version.
I see a lot of crash reports, with Settings not configured. Please visit Settings > [Apps] Pushover > “Settings” and fill in all the fields
What version are you running? Just pushed version 1.0.4 (this will indicate what version of the app your are running - there are two candiates now.)
It is not possible to enter this, when the app crashes right after install and when you press the restart app button, it crashes again immediately.
But really, the different “Pushover” apps should have different topics, @Kim_Kokholm , can you make a new one for your app? We are totally messing up this one with Off-topic.
