-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Youtube kiosk (Trending Music) addition Issue #8801 (Partial Solution) #9200
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Other than the two parts to revert this part of the code looks good, thanks!
dependencySubstitution { | ||
substitute module('com.github.TeamNewPipe:NewPipeExtractor') using project(':extractor') | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please undo the changes in this file, they are not meant to be committed! ;-)
Instead you should change the NewPipeExtractor
line in app/build.gradle
(699b4086398f1e0f412e178532e85c7a23fadf83 is the last commit in your extractor PR):
implementation 'com.github.CaptSzat:NewPipeExtractor:699b4086398f1e0f412e178532e85c7a23fadf83'
@@ -113,6 +113,7 @@ public class MainActivity extends AppCompatActivity { | |||
private static final int ITEM_ID_BOOKMARKS = -3; | |||
private static final int ITEM_ID_DOWNLOADS = -4; | |||
private static final int ITEM_ID_HISTORY = -5; | |||
private static final int ITEM_ID_TRENDING_MUSIC = -6; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think you need this, please revert the changes in this file ;-)
Closing as there has not been any activity after this PR was opened. Feel free to reopen after TeamNewPipe/NewPipeExtractor#955 is merged. This PR is really small anyway. |
What is it?
Description of the changes in your PR
I have added the Youtube Music Trending Kiosk. However I was unable to get a link to the YouTube music trending page so the Youtube Trending Music kiosk diusplays Trending feed until a fix can be found.
Fixes the following issue(s)
NewPipeExtractor: TeamNewPipe/NewPipeExtractor#955
Due diligence