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

Release 0.21.10 #7066

Closed
6 of 7 tasks
TobiGr opened this issue Sep 6, 2021 · 21 comments
Closed
6 of 7 tasks

Release 0.21.10 #7066

TobiGr opened this issue Sep 6, 2021 · 21 comments

Comments

@TobiGr
Copy link
Member

TobiGr commented Sep 6, 2021

APK for testing

From GitHub Actions: NewPipe_v0.21.10_RC3.zip
Corresponding pull request #7063

ToDo

App changelog

New

7- Added Show Channel Details where it's missing #6919
8- Allow choosing which types of search suggestions to show #3546
22- Added support for CSV+ZIP subscriptions (Google Takeout) #6882
24- Show popup menu when long pressing items in the video player's play queue #6955
27- Added option to directly open fullscreen when the main player starts #5459
28- Added darker splash screen for OS dark theme #7055

Improved

4- Changed dark theme colors to darker variant #6244
6- Replace UniversalImageLoader with Picasso #5928
9- Change "not available" image from PNG to vector format #6917
10- Provide mime type to file picker to gray out unselectable files #6820
23- Explain how to remove tab from main page #6965

Fixed

2- Catch errors while processing timestamp-links #6851
3- Added wrapping settings titles to the next line #6952
12- Re-add sharing of the content name with the "Share" command #6957
15- Fixed double tapping the replay button #7014
19- [Android TV] Fix long seekbar jumps when using a DPad #6985
25- Close audio effect control session properly #6993
26- Fixed some inconsistencies in the error panel #7004

Development

1- Added gradle parameter to skip formatKtLint and use in CI #6858
11- Using Eclipse temurin in GH actions #6995
14- Use ExoPlayer default values for buffers #6515
16- Use PackageInfoCompat.getSignatures(). #6634
18- Fixed build problems and updated kotlin #6986
21- Using integrated cache in actions/setup-java #7002

Translation

5- Use custom cancel string everywhere #6848
13- Added plural forms for download related strings #6930
17- Reword part of the bug report template #6990
20- Never use android.R.strings and remove unused add string #7024

Extractor changelog

New

3- Add invidious instance EduVid Tubus TeamNewPipe/NewPipeExtractor#712
5- Extract Uploader's Avatar in StreamInfoItems (YouTube and PeerTube) TeamNewPipe/NewPipeExtractor#723, TeamNewPipe/NewPipeExtractor#725
6- Make uploaderVerified available in StreamInfo TeamNewPipe/NewPipeExtractor#724

Fixed

2- [YouTube] Support csv and zip subscription import (Google Takeout) TeamNewPipe/NewPipeExtractor#709 TeamNewPipe/NewPipeExtractor#721
4- [SoundCloud] Update client id TeamNewPipe/NewPipeExtractor#722

Development

1- Bump gson from 2.8.7 to 2.8.8 TeamNewPipe/NewPipeExtractor#718

@TobiGr TobiGr mentioned this issue Sep 6, 2021
@TobiGr TobiGr pinned this issue Sep 6, 2021
@SameenAhnaf
Copy link
Collaborator

Not sure how to reproduce it. But app is crashing every now and then.

Exception

  • User Action: ui error
  • Request: ACRA report
  • Content Country: US
  • Content Language: en-US
  • App Language: en_US
  • Service: none
  • Version: 0.21.10
  • OS: Linux Android 11 - 30
Crash log

java.lang.ClassCastException: java.lang.Boolean cannot be cast to java.util.Set
	at android.app.SharedPreferencesImpl.getStringSet(SharedPreferencesImpl.java:312)
	at androidx.preference.Preference.getPersistedStringSet(Preference.java:1739)
	at androidx.preference.MultiSelectListPreference.onSetInitialValue(MultiSelectListPreference.java:205)
	at androidx.preference.Preference.onSetInitialValue(Preference.java:1614)
	at androidx.preference.Preference.dispatchSetInitialValue(Preference.java:1587)
	at androidx.preference.Preference.onAttachedToHierarchy(Preference.java:1311)
	at androidx.preference.Preference.onAttachedToHierarchy(Preference.java:1326)
	at androidx.preference.PreferenceGroup.addPreference(PreferenceGroup.java:249)
	at androidx.preference.PreferenceGroup.addItemFromInflater(PreferenceGroup.java:170)
	at androidx.preference.PreferenceInflater.rInflate(PreferenceInflater.java:345)
	at androidx.preference.PreferenceInflater.inflate(PreferenceInflater.java:157)
	at androidx.preference.PreferenceInflater.inflate(PreferenceInflater.java:109)
	at androidx.preference.PreferenceManager.inflateFromResource(PreferenceManager.java:216)
	at androidx.preference.PreferenceFragmentCompat.addPreferencesFromResource(PreferenceFragmentCompat.java:361)
	at org.schabi.newpipe.settings.ContentSettingsFragment.onCreatePreferences(ContentSettingsFragment.java:72)
	at androidx.preference.PreferenceFragmentCompat.onCreate(PreferenceFragmentCompat.java:160)
	at org.schabi.newpipe.settings.BasePreferenceFragment.onCreate(BasePreferenceFragment.java:28)
	at androidx.fragment.app.Fragment.performCreate(Fragment.java:2949)
	at androidx.fragment.app.FragmentStateManager.create(FragmentStateManager.java:475)
	at androidx.fragment.app.FragmentStateManager.moveToExpectedState(FragmentStateManager.java:278)
	at androidx.fragment.app.FragmentManager.executeOpsTogether(FragmentManager.java:2189)
	at androidx.fragment.app.FragmentManager.removeRedundantOperationsAndExecute(FragmentManager.java:2100)
	at androidx.fragment.app.FragmentManager.execPendingActions(FragmentManager.java:2002)
	at androidx.fragment.app.FragmentManager$5.run(FragmentManager.java:524)
	at android.os.Handler.handleCallback(Handler.java:938)
	at android.os.Handler.dispatchMessage(Handler.java:99)
	at android.os.Looper.loop(Looper.java:236)
	at android.app.ActivityThread.main(ActivityThread.java:8043)
	at java.lang.reflect.Method.invoke(Native Method)
	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:620)
	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1011)


@opusforlife2
Copy link
Collaborator

opusforlife2 commented Sep 6, 2021

Bug: With device locked to portrait, the app isn't rotating back to portrait upon tapping Back after the video is finished (and the Replay button is shown). Auto-fullscreen can be on or off.

I just reinstalled the auto-fullscreen APK and this bug exists there as well, which means it just slipped through testing.

If it's possible, this could be an opportunity to change current behaviour so that the app automatically goes back into portrait mode after a video is finished when the device is locked to portrait.

Edit: To be clear, on the current release, tapping Back after a video is finished does rotate the app back to portrait like it is supposed to. In RC1 (and the auto-fullscreen PR) you're stuck in landscape mode unless you manually tap the fullscreen button or exit the app entirely.

@litetex
Copy link
Member

litetex commented Sep 6, 2021

Not sure how to reproduce it. But app is crashing every now and then.
...

I also suffer from this bug when e.g. using the emulator and importing the (not migrated version of the) database over and over again.
I think that might be related to #3546...

Update: I did a bit of digging and it seems like when importing the settings the migration method is never called :/
Looks like #6495 doesn't do it's job...

@SameenAhnaf
Copy link
Collaborator

@opusforlife2 Shouldn't #5967 also be solved then?

@opusforlife2
Copy link
Collaborator

You have to enable autorotate to fix it.

That's not the case. As I said above, you can simply tap Back to rotate back to portrait.

But also as I've said above, I agree that this bug should be fixed properly if possible. Automatic return to portrait after the video is finished. OR, just stay in landscape fullscreen and exit directly to portrait upon tapping Back (instead of automatically going into a weird non-fullscreen but landscape hybrid.)

@litetex
Copy link
Member

litetex commented Sep 6, 2021

Created a fix for the failed migrations (but not only limited to them) here: #7068

@AudricV
Copy link
Member

AudricV commented Sep 8, 2021

The color of the action bar (back, home, recent apps) is not using the darker red when the system is in dark mode:

System.bar.dark.theme.mp4

Is there any way to change this?

@litetex
Copy link
Member

litetex commented Sep 9, 2021

@TiA4f8R
Checkout #7085 😄

@TobiGr
Copy link
Member Author

TobiGr commented Sep 10, 2021

Changes in RC2: NewPipe_v0.21.10_RC2.zip

@SameenAhnaf

This comment has been minimized.

@litetex
Copy link
Member

litetex commented Sep 12, 2021

@SameenAhnaf
This also happens currently (0.21.9) if you e.g. set the speed to 3x.
I think this is fine as more updates would cause more battery usage as described in #7071

@TobiGr
Copy link
Member Author

TobiGr commented Sep 13, 2021

Changes in RC3, which is likely the last RC NewPipe_v0.21.10_RC3.zip

  • Updated translations
  • Fixed restarting app correctly after importing an old app database

@XiangRongLin
Copy link
Collaborator

Note that the JitPack release for the extractor failed. TeamNewPipe/NewPipeExtractor#728

If it can't be resolved until this releases the commit hash has to be used instead of the version tag

@triallax triallax mentioned this issue Sep 17, 2021
3 tasks
@opusforlife2
Copy link
Collaborator

@TobiGr This regression is still unfixed: #7066 (comment)

@Pentaphon
Copy link

This regression is still unfixed:

I can't wait for that to be fixed. As somebody who is always using portrait mode, it is currently the most annoying part of using Newpipe. It keeps getting stuck until I back out of it.

@Stypox
Copy link
Member

Stypox commented Sep 19, 2021

cf9b482 is the first bad commit causing #7066 (comment)
...do I hate the player code, those changes have nothing to do with rotating the device when pressing back...
I will try to fix this later today

@opusforlife2
Copy link
Collaborator

@Stypox Maybe in the occasionally pinned features/bugs issue for devs, we could add an item: "Refactor the massive Player code into smaller, more manageable components".

@Stypox
Copy link
Member

Stypox commented Sep 19, 2021

@opusforlife2 regression fixed in 7667b2c (please confirm)
Btw, the cause of the problem was an if checking if the player is open before restoring the orientation, so it used to work because the player was not closed correctly. So this confirms that #5459 indeed correctly closes it ;-)
RC4: NewPipe_v0.21.10_RC4.zip

@triallax
Copy link
Contributor

Is anything blocking this issue getting closed? The corresponding PR has already been merged.

@Pentaphon
Copy link

Is anything blocking this issue getting closed? The corresponding PR has already been merged.

I guess the last commit has to be confirmed and it's ready to ship.

@opusforlife2
Copy link
Collaborator

Wow that was fast. Yes indeedy! The regression has been fixed. Release the ostriches!

@TobiGr TobiGr closed this as completed Sep 20, 2021
@TobiGr TobiGr unpinned this issue Sep 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

9 participants