Skip to content

Improve new stream notifications - #13691

Open
Isira-Seneviratne wants to merge 7 commits into
TeamNewPipe:devfrom
Isira-Seneviratne:Improve-notifications
Open

Improve new stream notifications#13691
Isira-Seneviratne wants to merge 7 commits into
TeamNewPipe:devfrom
Isira-Seneviratne:Improve-notifications

Conversation

@Isira-Seneviratne

Copy link
Copy Markdown
Member

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

  • Updated new stream notifications to show the stream thumbnail.
  • Improved notification ordering using the upload date.
  • Limited new stream notifications to 10 per channel to avoid creating too many notifications, and updated the oldest allowed date to last week from the current date.

Before/After Screenshots/Screen Record

Android 12

Screenshot_20260717_061413 Screenshot_20260717_061425 Screenshot_20260717_061436

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

@github-actions github-actions Bot added the size/medium PRs with less than 250 changed lines label Jul 17, 2026
@TobiGr TobiGr added the new stream notification Issue is related to notification shown for new streams label Jul 17, 2026
@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 21, 2026
@sonarqubecloud

Copy link
Copy Markdown

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

Labels

new stream notification Issue is related to notification shown for new streams size/large PRs with less than 750 changed lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants