Slack Notifications

[ezekielj221]

V 1.9.1

Windows Server 2012 R2

Test Notification goes through fine but never get notifications from requests

[mgoodsell91]

+1 Ive got exactly the same issue.

[rrbailey89]

It was working for me but is no longer working

[tidusjar]

This is for the request notification? That goes to the admin?

[rrbailey89]

Yes. It's weird it used to work for me. I disabled the notification for a few days then re enabled it in settings and now I don't get it. @tidusjar

[tidusjar]

Who is requesting, the user? Since if an admin requests, we don't send a notification

[rrbailey89]

@tidusjar seems any user. I just disabled user authentication and password entered a users username to "login" requested a movie and got no notification from slack.

[tidusjar]

So in the backend we keep a list of "notifications". When you enable a notification it get's added to the list and when we trigger a notification (User request) we send out a message to everyone in the list.

A bug I found is when pressing the "Test" button it would always remove it from that list after.

I have now fixed that, are you willing to try out a dev build to confirm the fix?

[rrbailey89]

sure thing @tidusjar what should I do to get it?

[tidusjar]

https://ci.appveyor.com/project/tidusjar/requestplex/build/artifacts :)

[rrbailey89]

@tidusjar maybe i updated wrong .. I copy and pasted over the installation and ran nssm restart PlexRequests.net but now when I try to browse to the site it says connection refused.

[tidusjar]

You should just be able to copy and paste. Try running the .exe manually?

On Sun, 18 Sep 2016 at 5:44 pm, Serena notifications@github.com wrote:

@tidusjar https://github.com/tidusjar maybe i updated wrong .. I copy
and pasted over the installation and ran nssm restart PlexRequests.net
but now when I try to browse to the site it says connection refused.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#536 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGVaLIbf1VJLWx_FhkUw2V1lE2or2d4pks5qrWpWgaJpZM4J9QlF
.

Thanks,
Jamie

[rrbailey89]

this is the error that comes up when I try running directly from the .exe http://pastebin.com/J18UUrnw

[tidusjar]

Did you unblock the zip before extracting ?

Thanks,
Jamie

[rrbailey89]

@tidusjar ... 😵 .. no haha..

[rrbailey89]

@tidusjar it seems to be working now :) with notifications

[rrbailey89]

a new possibly unrelated error though

System.NullReferenceException: Object reference not set to an instance of an object. at System.Linq.Enumerable.d__941.MoveNext() at System.Linq.Buffer1..ctor(IEnumerable1 source) at System.Linq.Enumerable.ToArray[TSource](IEnumerable1 source) at PlexRequests.Services.Jobs.CouchPotatoCacher.QueuedIds() in C:\projects\requestplex\PlexRequests.Services\Jobs\CouchPotatoCacher.cs:line 95

[tidusjar]

Excellent!

I've also made the fix in all of the notifications since they all had the
same problem. I'm thinking about pushing a small release out in a few hours
:)

On Sun, 18 Sep 2016 at 5:56 pm, Serena notifications@github.com wrote:

@tidusjar https://github.com/tidusjar it seems to be working now :)
with notifications

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#536 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGVaLHVWAF-MR9aV_yVQTiK504MuoXrhks5qrW1IgaJpZM4J9QlF
.

Thanks,
Jamie