-
Notifications
You must be signed in to change notification settings - Fork 30
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
Feature Request: Discord Notifications for Unmatched Assets #194
Comments
The reason this was never added was because not everybody has a small list and this could be thousands of entries posted to discord (mines for example). Even if this was an option to add and to make it make sense for all involved. What would be your idea for this? |
My idea would be to have a discord notification similar to what I get from poster_renamerr. Wouldn't discord just turn the message into a downloadable file when it's over the character limit? There could be a disclaimer to only enable this notification if you know what you're doing as it could cause very large messages. |
Unfortunately when dealing with discord and it's webhooks if the message is over the character limit it will simply just kick back an error and do nothing. |
In there anyway to integrate with a pastebin type of service so it just posts a link? |
or maybe github gists like the notifiarr bot does. https://docs.github.com/en/rest/gists/gists?apiVersion=2022-11-28#create-a-gist |
Not that I'm aware of. However if somebody wants to submit a PR in order to make this addition I'm not opposed to it especially if it was a configurable option. |
Maybe something worth looking into when I get the time. My life is very hectic right now I haven't even opened my laptop in quite a while. But I'm for sure not opposed to it. However if somebody gets to it before I do that'd be great. |
It would be super helpful to be able to get a discord notification for unmatched assets rather than have to regularly check the log. Since it's regularly a small list for most, being able to quickly reference the notification to stay on top of them would be very beneficial.
The text was updated successfully, but these errors were encountered: