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

Youtube kiosk (Trending Music) addition Issue #8801 (Partial Solution) #9200

Closed
wants to merge 3 commits into from

Conversation

CaptSzat
Copy link

@CaptSzat CaptSzat commented Oct 26, 2022

What is it?

  • Bugfix (user facing)
  • Feature (user facing)
  • Codebase improvement (dev facing)
  • Meta improvement to the project (dev facing)

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

Copy link
Member

@Stypox Stypox left a 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')
}
}
Copy link
Member

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;
Copy link
Member

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 ;-)

@SameenAhnaf SameenAhnaf added feature request Issue is related to a feature in the app youtube Service, https://www.youtube.com/ GUI Issue is related to the graphical user interface labels Feb 12, 2023
@Stypox
Copy link
Member

Stypox commented Mar 30, 2024

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.

@Stypox Stypox closed this Mar 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request Issue is related to a feature in the app GUI Issue is related to the graphical user interface youtube Service, https://www.youtube.com/
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make a YouTube music trending Kiosk
3 participants