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

Remove private api stuff #684

Merged
merged 7 commits into from
Nov 21, 2019
Merged

Remove private api stuff #684

merged 7 commits into from
Nov 21, 2019

Conversation

bastimeyer
Copy link
Member

@bastimeyer bastimeyer commented Nov 20, 2019

Resolves #681

This removes everything that is related to the private Twitch API endpoints which are now blocked when requested with third party client-IDs.

As mentioned in #681, channel panels are still available, but only via Twitch's client-id and since Twitch's website has moved to their new private API based on GraphQL a long time ago, it is questionable how long channel panels will still be available on the old private API. I don't really want to use Twitch's client-id in the Twitch GUI, so I'm removing this feature for now.

Also, followed games may be available on the kraken v5 API (undocumented, similar to the old endpoints on the private API), but since unfollowing games doesn't seem to be possible, I don't want to keep this feature.

Should any of these restricted API endpoints become available on kraken (unlikely, since it's been deprecated) or on helix, I will try to re-implement them.


Checklist

  • Remove subscription list
    • Remove UserSubscriptionsRoute
    • Remove SubscriptionItemComponent
  • Remove hosted streams list
    • Remove UserHostedStreamsRoute
    • Clean up StreamItemComponent
  • Remove followed games
    • Remove UserFollowedGamesRoute
    • Remove FollowGameComponent
    • Clean up GameItemComponent
  • Remove channel panels
    • Remove masonry-layout dependency
    • Clean up ChannelIndexRoute
  • Clean up localizations
  • Add fixes for homepage settings of removed routes
  • Restructure main menu
    • Remove unused hotkeys

@codecov
Copy link

codecov bot commented Nov 20, 2019

Codecov Report

Merging #684 into master will increase coverage by 0.07%.
The diff coverage is 81.81%.

@@            Coverage Diff             @@
##           master     #684      +/-   ##
==========================================
+ Coverage    85.3%   85.38%   +0.07%     
==========================================
  Files         466      444      -22     
  Lines       14059    13702     -357     
  Branches     1257     1238      -19     
==========================================
- Hits        11993    11699     -294     
+ Misses       1972     1912      -60     
+ Partials       94       91       -3

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

Successfully merging this pull request may close these issues.

Panels API gone
1 participant