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

Fix some deprecations and other warnings #750

Merged
merged 4 commits into from
Nov 10, 2023

Conversation

Luna712
Copy link
Contributor

@Luna712 Luna712 commented Nov 2, 2023

  • Replace deprecated onBackPressed()
  • Fixes some AndroidManifest warnings
    • Use android:dataExtractionRules for Android 12+
    • Add tools:ignore for ScopedStorage and QueryAllPackagesPermission
    • Make VideoDownloadRestartReceiver not exported (fixes "
      Exported receiver does not require permission") - if this was incorrect I can revert this part
    • Bump tools:targetApi
  • Enable android:enableOnBackInvokedCallback for predictive back gestures on Android 13 (if enabled in developer options) and Android 14 (also fixes "OnBackInvokedCallback is not enabled for the application." which is commonly seen in logcat)

* Replace deprecated onBackPressed()
* Fixes some AndroidManifest warnings
  * Use android:dataExtractionRules for Android 12+
  * Add tools:ignore for ScopedStorage and QueryAllPackagesPermission
  * Make VideoDownloadRestartReceiver not exported (fixes "
Exported receiver does not require permission") - if this was incorrect I can revert this part
  * Bump tools:targetApi
* Enable android:enableOnBackInvokedCallback for predictive back gestures on Android 13 (if enabled in developer options) and Android 14 (also fixes "OnBackInvokedCallback is not enabled for the application." which is commonly seen in logcat)
@IndusAryan
Copy link
Contributor

lol, I was just minutes away from creating similar backpressed upgrade pr

@fire-light42
Copy link
Collaborator

@Blatzar

@Blatzar
Copy link
Contributor

Blatzar commented Nov 2, 2023

Very nice to see this, maybe we can implement some cool predictive back gestures after this 🙏

@Luna712 Luna712 requested a review from Blatzar November 2, 2023 22:47
Copy link
Contributor

@Blatzar Blatzar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Testing looks good 👍

@fire-light42
Copy link
Collaborator

This wont be merged until the bottom color mismatch is fixed
image

Copy link
Collaborator

@fire-light42 fire-light42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🥳 Very good job, nice pr as always. Reply with 👍 or react and I will merge it

@fire-light42 fire-light42 merged commit 3adf036 into recloudstream:master Nov 10, 2023
2 checks passed
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

Successfully merging this pull request may close these issues.

None yet

4 participants