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 signalr #11

Merged
merged 4 commits into from Jan 4, 2018

Conversation

Projects
None yet
2 participants
@brunojuliao
Contributor

brunojuliao commented Jan 4, 2018

It seems to be working fine.

Few observations:

  • I noticed on my tests that it takes +- 30s to trigger the SignalR command (because TwitchAPI code seems to be comparing the number of subs from time to time).

  • I only tested twitch, but mixer should be working as well :)

I had to make some research, as this was my first time working with SignalR, and on references below are the 2 links that helped me make the changes in this "pull request".

References:
https://www.codeproject.com/Articles/1204852/ASP-NET-Core-SignalR-Core-alpha
https://webactivedirectory.com/asp-net/asp-net-use-url-content-from-razor-to-resolve-relative-urls/

@csharpfritz csharpfritz merged commit dc4c5ec into csharpfritz:feature-signalr Jan 4, 2018

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