Skip to content

Conversation

TobiGr
Copy link
Contributor

@TobiGr TobiGr commented Jul 21, 2025

What is it?

  • Bugfix (user facing)

To Do

  • Update extractor
  • Remove debug conditions
  • Use correct id for live trending kiosk
  • Polish strings

Description of the changes in your PR

User-facing

Changes the combined YouTube Trending kiosk the the Live Trending kiosk. For more info see #12445.
Prevents / fixes crash for users which have the YT trending page as a separate kiosk. The kiosk has been kept, as it is still working for now.

Dev-facing

Separates settings migrations from UI informing the user about the changes and the code that handles the UI prompts.
Allow queuing multiple user facing UI actions for SettingsMigrations.

Before/After Screenshots/Screen Record

Soon

Fixes the following issue(s)

Closes #1805, closes #12445

Relies on the following changes

Changes on the extractor side which have not yet been completed.

APK testing

The APK can be found by going to the "Checks" tab below the title. On the left pane, click on "CI", scroll down to "artifacts" and click "app" to download the zip file which contains the debug APK of this PR. You can find more info and a video demonstration on this wiki page.

Due diligence

@TobiGr TobiGr added the youtube Service, https://www.youtube.com/ label Jul 21, 2025
@TobiGr TobiGr added this to v0.28.x Jul 21, 2025
@github-project-automation github-project-automation bot moved this to Todo in v0.28.x Jul 21, 2025
@TobiGr TobiGr moved this from Todo to In Progress in v0.28.x Jul 21, 2025
@github-actions github-actions bot added the size/medium PRs with less than 250 changed lines label Jul 21, 2025
@TobiGr TobiGr force-pushed the yt-trending-migration branch 3 times, most recently from cb3711d to 2b6b8b7 Compare July 22, 2025 13:46
@ShareASmile ShareASmile added feature request Issue is related to a feature in the app requires extractor change This issue requires a change to the extractor labels Jul 23, 2025
@TobiGr TobiGr changed the base branch from dev to release-0.28.0 July 24, 2025 15:46
@github-actions github-actions bot added size/large PRs with less than 750 changed lines size/medium PRs with less than 250 changed lines and removed size/medium PRs with less than 250 changed lines size/large PRs with less than 750 changed lines labels Jul 28, 2025
@Stypox
Copy link
Member

Stypox commented Jul 28, 2025

A command that can be used to test the migration:

adb shell run-as org.schabi.newpipe.debug.pr12450 'sed -i '"'"'s#<int name="last_used_preferences_version" value="8" />#<int name="last_used_preferences_version" value="6" />#'"'"' shared_prefs/org.schabi.newpipe.debug.pr12450_preferences.xml' && adb shell run-as org.schabi.newpipe.debug.pr12450 'sed -i '"'"'s#\]}</string>#,{\&quot;tab_id\&quot;:5,\&quot;service_id\&quot;:0,\&quot;kiosk_id\&quot;:\&quot;Trending\&quot;},{\&quot;tab_id\&quot;:5,\&quot;service_id\&quot;:1,\&quot;kiosk_id\&quot;:\&quot;Top 50\&quot;}]}</string>#'"'"' shared_prefs/org.schabi.newpipe.debug.pr12450_preferences.xml'

@Stypox Stypox mentioned this pull request Jul 31, 2025
11 tasks
@github-actions github-actions bot added size/large PRs with less than 750 changed lines and removed size/medium PRs with less than 250 changed lines labels Jul 31, 2025
@Stypox Stypox marked this pull request as ready for review July 31, 2025 21:33
TobiGr and others added 7 commits July 31, 2025 23:43
You can use this command to test instead:

adb shell run-as org.schabi.newpipe.debug.pr12450 'sed -i '"'"'s#<int name="last_used_preferences_version" value="8" />#<int name="last_used_preferences_version" value="6" />#'"'"' shared_prefs/org.schabi.newpipe.debug.pr12450_preferences.xml' && adb shell run-as org.schabi.newpipe.debug.pr12450 'sed -i '"'"'s#\]}</string>#,{\&quot;tab_id\&quot;:5,\&quot;service_id\&quot;:0,\&quot;kiosk_id\&quot;:\&quot;Trending\&quot;},{\&quot;tab_id\&quot;:5,\&quot;service_id\&quot;:1,\&quot;kiosk_id\&quot;:\&quot;Top 50\&quot;}]}</string>#'"'"' shared_prefs/org.schabi.newpipe.debug.pr12450_preferences.xml'
Currently translated at 12.7% (95 of 748 strings)

Co-authored-by: fool <[email protected]>
@Stypox Stypox force-pushed the yt-trending-migration branch from 08b241f to 5aefa4a Compare July 31, 2025 21:43
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.

I changed the trending names as agreed over Matrix and tested migrations again, everything seems to work. Thanks!

@Stypox Stypox merged commit 4d97a76 into release-0.28.0 Jul 31, 2025
5 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in v0.28.x Jul 31, 2025
@AudricV AudricV changed the title Add SettingsMigration to change YouTube trending kiosk tab [YouTube] Support new kiosks and add migration to change YouTube trending kiosk tab Jul 31, 2025
@AudricV AudricV changed the title [YouTube] Support new kiosks and add migration to change YouTube trending kiosk tab [YouTube] Support new kiosks and add migration to change trending kiosk tab Jul 31, 2025
@TobiGr TobiGr deleted the yt-trending-migration branch August 1, 2025 06:06
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 requires extractor change This issue requires a change to the extractor size/large PRs with less than 750 changed lines youtube Service, https://www.youtube.com/

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants