Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[REQ] Discord Webhook Support #739

Closed
karbowiak opened this issue Dec 8, 2016 · 6 comments
Closed

[REQ] Discord Webhook Support #739

karbowiak opened this issue Dec 8, 2016 · 6 comments

Comments

@karbowiak
Copy link

Hi there!

Discord has a Webhook just like Slack does now, and it's even Slack compatible if you add /slack to the end of the webhook url.

Sadly tho, PlexRequests.Net doesn't obey the url, and tries to use it for Slack.
Any chance of a catch to fix so that if the url contains Discord in it, it'll prod Discord instead of Slack? :)

@tidusjar
Copy link
Member

tidusjar commented Dec 8, 2016

Interesting,

Personally, i'd rather add a new section rather than hacking the Slack notification to do this.

But Discord support shall be done!

@karbowiak
Copy link
Author

Woohoo \o/

Either way would work really :)

More info available here: https://support.discordapp.com/hc/en-us/articles/228383668-Intro-to-Webhooks
and https://discordapp.com/developers/docs/resources/webhook (Probably most interesting)

@tidusjar
Copy link
Member

tidusjar commented Dec 8, 2016

Thanks!

@Magikarplvl4 Magikarplvl4 added this to the Release v2.1 - notification update milestone Dec 8, 2016
@Magikarplvl4
Copy link
Contributor

Planned in the big notification update

@Merckle
Copy link

Merckle commented Dec 21, 2016

Id love to see this working as well. I was playing around with it but couldnt seem to ever get it working with out getting errors

tidusjar pushed a commit that referenced this issue Jan 17, 2017
tidusjar pushed a commit that referenced this issue Jan 17, 2017
@tidusjar tidusjar modified the milestones: Release v2.1, Release v2.2 Jan 17, 2017
@tidusjar tidusjar removed the planned label Jan 17, 2017
@Magikarplvl4
Copy link
Contributor

It is now live in the 2.1 version!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants