What's new
  • Visit Rebornbuddy
  • Visit Resources
  • Visit API Documentation
  • Visit Downloads
  • Visit Portal
  • Visit Panda Profiles
  • Visit LLamamMagic

Pushover Notifications - Trinity

coloradoderek

New Member
Joined
Oct 17, 2014
Messages
17
Reaction score
0
Good Morning.

Is anyone else having problems using notifications?

I have setup my Pushover API key in the Trinity Settings but it doesn't send me notifications when it stores a Legendary.

This used to work, then I re-installed my computer and since then it hasn't worked. Not sure if its a problem on my end, or a setting I haven't checked somewhere, or if the newest version of Demonbuddy is having issues.

I'll continue to try and test just was wondering if anyone else is using notifications and is having problems.

I can also search the code to see what's up but its hard to figure out exactly what is happening.

Thanks!
Derek Conlon
 
Just an update: I have updated to the most recent version of trinity and checked all settings looking for something that might have been wrong and confirmed my bot did in fact store something in the stash, and no notifications were sent.

I also tested my pushover with another app and it works too.

thanks!
 
I went into the Notifications.cs file and changed the token for Pushover in there to be my own token for my own pushover account.
Will update if this works as soon as an Item drops I want to keep :P
 
doesn't seem this worked either.

will have to look into the code further and see what I can figure out.
 
SOLVED:

In NotificationManager.cs:

Change:
Line (177ish) : "&priority=" + HttpUtility.UrlEncode(notice.Priority.ToString());

This is doing this: &priority=Emergency when you look at the URL variable after its fully built.

This is the wrong syntax. The priority needs to be a number, not a word. (Pushover: API)

I changed the line mentioned above to this: "&priority=0";

and this solved the problem.
You don't want it to be an Emergency anyway as it will spam your device with notifications until you acknowledge it.

Thanks,
Derek Conlon
 
Back
Top