-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
[Streams] Game allow list for stream alerts #4174
base: V3/develop
Are you sure you want to change the base?
[Streams] Game allow list for stream alerts #4174
Conversation
Quick look at api docs to refresh myself tells me that Youtube is a definite no. Picarto actually does have categories (which are given as a string when retrieving a stream but have more data beyond that if grabbing the list of categories and thus the category name can be validated) that could be used to filter but this assumes that is something that can change from stream to stream, which I'm ultimately not sure on simply because I'm not at all familiar with Picarto. If anyone happens to have insight into this, feel free to let me know |
Twitch and Smashcast support have been implemented. Haven't touched Picarto for the reasons I mentioned in #4174 (comment) Otherwise, this needs testing (I haven't really done much testing of this myself) |
So to start after a full day of testing...
|
The issue with |
i'm getting it with
|
yeah, that would be the result of copying and pasting and changing what needed to be changed lol Which is now fixed |
So, testing have been done, no more issues found, and it sends the correct game from twitch and smashcast. it just takes longer to get notified than i though, about 3 - 5 minutes to wait, but other than that, it works great, i really like this. also, just to leave about this:
|
yeah, that would be a bug |
Merge conflicts have been taken care of, though this will definitely require testing to ensure nothing broke by fixing merge conflicts |
…/streamalert_gamefilter
I can't get this function to work, and the help menu isn't very straightforward. I tried running Furthermore, The bot isn't announcing anything at all. Even without a filter. So i'm not sure of the function as it stands. |
@sp00nd It's a subcommand. The help should tell you that it's |
I tried that first, and figured it was wrong as it was spitting back
|
yeah, it looks like that function got removed when I fixed merge conflicts. I'll get this fixed |
Ok, try updating and then that issue should be fixed |
I ran Upon running the command, it is no longer erroring out. However, I am getting |
How exactly should the game name be when it refers to |
Exactly this. Copy and paste the game name from the game's page on Twitch |
This is exactly what I have been doing, and it is not recognizing them. Like I mentioned, |
Is there any way I can provide more logging on this error? I have not been able to get any category working from what i've seen. Used a fresh API key also when troubleshooting this. |
I've actually figured out how to go about using IGDB's API to potentially just allow a search of their API for the user's desired game, which would potentially allow the user to do something like |
Is there an opportunity anywhere in here to add support for filtering by words in stream title and by tags as well? More context: #2356 (comment) |
Type
Description of the changes
Game filter for stream alerts. Should ultimately support Twitch and Smashcast unless anything has changed with regards to the idea in Youtube or Picarto's APIs. Replacement for #2433, fixes #2356 and #3654