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

Updated to Android SDK 35 Compatibility and Fixed Sensor Initialization #177

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

HasanKoc33
Copy link

Summary

This PR addresses two key issues in the project:

  1. Android SDK 35 Compatibility:

    • Updated compileSdkVersion and related Gradle configurations to support Android 35.
    • Bumped the Android Gradle Plugin (AGP) version to 8.1.0 as required.
  2. Fixed Sensor Initialization Issue:

    • Resolved an issue in GyroscopeStreamHandler.kt where the sensor variable was causing a build failure. Updated its declaration to:
      private val sensor: Sensor? = sensorManager.getDefaultSensor(Sensor.TYPE_GYROSCOPE)

Testing

  • Verified compatibility with Android SDK 35.
  • Tested gyroscope functionality to ensure the sensor data is received correctly.

Let me know if further adjustments are needed. Thank you!

@Sahil-Simform
Copy link
Collaborator

Hi @HasanKoc33 ,
Thank you so much for your contribution! We appreciate your effort.
Can you please provide information as to why we need to change compile SDk to 35?

@Sahil-Simform Sahil-Simform added the more-info-required More information has been requested label Jan 9, 2025
@HasanKoc33
Copy link
Author

Upon upgrading to the latest version of Flutter, I noticed that the target SDK was updated to 35, which caused compatibility issues with the plugin. This observation led me to identify the target SDK update as the root cause of the error.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
more-info-required More information has been requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants