search

fullphat / snarl

When { stuff_happens(); tell_me() }
http://snarl.fullphat.net
57 stars 2 forks source link

GNTP: url provided icons in app not shown in notification #68

Open iMelsom opened 5 years ago

iMelsom commented 5 years ago

This issue is not usage breaking, but it's a feature that is nice to have working.

It seems that when an incomming GNTP notification arrives, the URL to the app or notification icon is ignored and replaced with the default icon.

Edit: The app/notification icons are ignored on app registration, and thus not used when notifications using an app is used.

fullphat commented 5 years ago

Just for completeness, can you paste the GNTP request you're making?

iMelsom commented 5 years ago

Hi. As I tried to use the snarl console to sniff out what comes in, I discovered that typing 'help' provided me with the message: UnknownCommand: "help"..

I'm using this Net_Growl class: http://growl.laurent-laville.org/manual/current/en/netgrowl-book.html

Logs from registering and recieving GNTP notifications: snarlosd.log GNTP_2aa2de5f-a1ef-41ec-8a4b-b25ec56957ff.txt GNTP_50209d6c-395a-4b4e-88bf-0a534f76379c.txt

fullphat commented 5 years ago

I assume it's this icon that's not being displayed?

http://intranett/dev/xiame/growlIM/Icons/SC.png

Does it definitely exist? That is, can you display it in a browser if you go to that URL?

Can you run snarl with the --log switch enabled, then do the test, and then send me all *.debug files that are created in the folder?

I've tested this using growlnotify and it works ok* - here's what I did:

growlnotify.com /a:Foo /ai:https://fullphat.net/callback/icons/system/misc-alien.png test

And then:

growlnotify.com /a:Foo /i:https://fullphat.net/callback/icons/system/misc-chair.png test

Beta 1 doesn't support the GNTP application-icon parameter (now fixed in Beta 2), but otherwise the result is as expected (application is registered and the notification appears with a chair icon.

iMelsom commented 5 years ago

Hi. Yes I can do that, but It may take a while before I'm back where I am able to due to work rota. I'l l update with logs, and status as soon as I can :)

iMelsom commented 5 years ago

Hi. I have tested it, and it now shows the app specific Icons. It do not, however use notification spesific. So if the apps speicfies different notifications types, that each has a separeate icon, it only uses the main icon for the app.