OrangeDrangon / android-messages-desktop

Android Messages as a Cross-platform Desktop App
Other
321 stars 27 forks source link

Display app name instead of id in notification toast on Windows #354

Closed vanowm closed 2 years ago

vanowm commented 2 years ago

Windows 10 no longer require set appId for notifications, we can set actual title instead: app.setAppUserModelId("Android Messages"); https://github.com/OrangeDrangon/android-messages-desktop/blob/3bb94a30b53a7b98f7f7daa4e33d2961aaec6c95/src/background.ts#L71

OrangeDrangon commented 2 years ago

According to this it is not required but has a performance benefit for setting it.

https://docs.microsoft.com/en-us/windows/win32/shell/appids?redirectedfrom=MSDN

I am sure there is a more correct place to set the name of the application for the os. That is specifically meant to be a unique application id from my understanding.

OrangeDrangon commented 2 years ago

https://www.electronjs.org/docs/latest/api/app#appname This may outline how to do that.

vanowm commented 2 years ago

My point was whatever is set by that function will be displayed as title of the notification toast, which currently is pw.kmr.android-messages-desktop: image

So if that line omitted, it displays electron.app.Electron: image

Or with app.setAppUserModelId("Android Messages"); it displays: image

I don't know how critical it is to follow "proper" format, but at least on Windows 10 21H2 it's not critical.

OrangeDrangon commented 2 years ago

I understand your point. I am saying that I am pretty sure that is not the intended way to associate a plain text name to a given application.

This comment outlines what I believe to be the intended way to get a plain text name. Though I may be wrong.

https://github.com/electron/electron/issues/5313#issuecomment-219960855

innesst commented 2 years ago

I wonder if this has something to do with the issue people have brought up that this application does not show up in the Windows 10 Notifications Setting. The notifications come through and I can adjust using Google's options, but there is no way to adjust them using the built in Windows settings (e.g. Number of notifications visible in action center, etc.).

I have Android Messages in my start menu like is mentioned in electron comment 5313, but the notification is still named pw.kmr.android-messages-desktop. Perhaps Windows only checks to adjust the notification banner if the application is properly showing up in the notification area of Windows settings.

innesst commented 2 years ago

This may have useful information / links https://stackoverflow.com/questions/63933737/electron-app-not-showing-in-notification-settings

330

OrangeDrangon commented 2 years ago

please download the binary the github action produces and let me know if this works.

innesst commented 2 years ago

It didn't work - but I have somethings I will try and report back. I was able to get the name on the notification to change.... I'll putz around and let you know.

OrangeDrangon commented 2 years ago

This is fixed in #368. Please download and test.