-
-
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
Release v0.16.2 #2314
Comments
On the last stable, in the What's New page only shows some videos and then just sits there with the loading icon. Is that a known bug? |
version in RC1 is reported incorrectly as 0.16.0 |
not entirely sure which steps led to this, hence unable to replicate, but sharing just in case: Exception
Crash log
|
Exception
Crash log
|
@TobiGr @theScrabi For me it's pretty strange to not have the player that works inside a window with comments, description, etc. |
@avently Also, many things have been added. They just aren't front end things. Better downloaders, more stable players, and other things have all been added. These things aren't really visible to the end user, but make for a more stable experience, which imo is better than willy nilly adding new features. |
Everyone can use my code from that PR without restrictions. I only said the PR shouldn't be reopened since no decisions were made for a long time. If a strategy doesn't work, make a new strategy:) Anyway, I have been using the code for years but not without problems. They are not critical and can be fixed easily. The idea behind the code is good but result could be better. And one large PR is a bad idea - multiple small pull requests is better in this case. |
From that PR,
Sorry for the misunderstanding. I thought that "create your own" meant that people couldn't use it. |
I'm experiencing a problem in playing specific videos on the background player. An example of a video like this: It happens quite often. I tried forcing shutdown of the app and trying to load it again, I tried even uninstalling the app and downloading it again, and the same song still won't load (but only on background mode). I know with a problem so unspecific like this I'm not helping you much. |
I have it running in background now. |
Thanks you all for testing! @Anotherlife This is an old bug and will be fixed in #2309 (not in this release) @ZMRH I couldn't reproduce this either. Maybe on of these annoying Android 5 bugs? :) @Brimont This is an extractor exception. But we didn't touch any related code. So I guess this was either a temporary bug or it exists in other versions, too. @avently It's good to hear from you! Welcome back. An embedded player is definetely on our list.
That's good to hear. I also thought we weren't allowed to reuse the code. I guess parts of it are very useful for us.
Yes, splitting this up into small PRs makes much sense and reviewing, testing and QA faster. @shmil111 Yes, me too. But this is not related to this PR as I have problems with it on both versions. We should track this down in #2336 and #2337. |
@TobiGr could you make a new issue with todo list related to an embedded player implementation? |
Thanks to @Stypox, we can ship two more fixes with 0,16.2:
Here is the new RC: newpipe_0.16.2_rc2_debug.apk.zip |
@avently Sure, but this might take a while as I would like to release the new version first and then review the three large PRs. I hope, I have some time in between, because I'd like to take a look at the old code, too. @theScrabi, @mauriciocolli In case you have some spare time, feel free to contribute to the list. |
I didn't test it with v0.16.2 too, but I witness issues with some SoundCloud streams (not all of them) since yesterday (#2344 & TeamNewPipe/NewPipeExtractor#166). It may be release-blocking? Exception
Crash log
|
The SoundCloud issues are there with 0.16.2 RC2 too. |
We didn't change anything related to SoundCloud in this release. It looks like the SoundCloud API changed. a bunch of SoundCloud tests are failing (see extractor). As long as nobody takes a look at it, this will be unfixed in 0.16.2. But be assured, that we will dive into it soon. |
Add a recommendation tab |
@Paris302 You could open an issue for that |
We should release bug fixes from time to time... :)
Please test and share new crashes and bugs witch us below. Corresponding pull request: #2312
newpipe_0.16.2_rc2_debug.apk.zip (by @TobiGr)
Changelog
Improvements
Fixed
ToDo
The text was updated successfully, but these errors were encountered: