chore: Merge branch dev to main

[github-actions]

This pull request will Merge branch dev to main.

Before merging this PR

• Pull translations from Crowdin

[0xrxL]

@LisoUseInAIKyrios I think 19.50 will be the last version for this year, maybe it's better to add a support for this instead?

[LisoUseInAIKyrios]

19.50 has bugs in the miniplayer. 19.47 is better.

[LisoUseInAIKyrios]

Need to merge strings, otherwise patching will show warnings.

[oSumAtrIX]

Is there a way to reduce the fragility if the translations system? Errors like this shouldn't be easy to make

[LisoUseInAIKyrios]

Yes, AddResourcePatch can check and do not add translations from a non english (non /values/strings.xml file), unless there also is an English string that has the same key.

That would fix these harmless warnings, where a string was deleted but since Crowdin has not been synced the localized languages still have the string:
WARNING: warn: removing resource app.revanced.android.youtube:string/revanced_spoof_video_streams_about_android_vr_summary without required default value.

[oSumAtrIX]

The AddResourcesPatch should not fulfill specific requirements that don't apply to the Android specifications because it works as a generic patch that adds resource files. So unless it is invalid behavior to add localized strings unless the original exists, probably adding the behavior to not add the localized string may go against the specs

[LisoUseInAIKyrios]

Adding localized strings without a default does go against android standards.

The warning shown above is from resource compilation where it fixes the problem by not including the localized string. Add resources could avoid the warning by doing the check itself and not adding.

Or alternatively always merge crowdin strings before merging to main. Dev releases will still show the warning but it's dev so who cares.

[oSumAtrIX]

In that case I agree AddResourcesPatch should explicitly handle this invalid behavior