sentry-java icon indicating copy to clipboard operation
sentry-java copied to clipboard

Please specify minSdkVersion for android

Open SerggioZS opened this issue 1 year ago • 4 comments

Problem Statement

Avoid making us search for the minSDK of the dependencies sentry uses in each version iteration. We are using libraries sentry also uses, but we need to use specific versions to support older devices.

Solution Brainstorm

Just put the minSdkVersion in the changelog on each version release for sentry, sentry AGP, etc. Thank you!

SerggioZS avatar Feb 26 '24 15:02 SerggioZS

hey @SerggioZS we rarely bump our minSdk version, and when we do, it's usually a major version bump of the SDK and we explicitly state it in the changelog, like here https://github.com/getsentry/sentry-java/releases/tag/7.0.0.

Also we have this table (which I see is outdated, but we'll get this fixed): https://github.com/getsentry/sentry-java#sentry-sdk-for-java-and-android

Is that not enough, or what's your use-case exactly?

romtsn avatar Feb 26 '24 16:02 romtsn

Hi @romtsn

It would be nice to have that table updated for each version of Sentry. Also do the same thing for Sentry Android Gradle Plugin AND Sentry implementation via BOM. I think it's important to know which min SDK version of Android are enforced in each version and implementation of Sentry.

My use case now it to know wich version of Sentry works in a project with minSDK 19 and on Android 4.4 without making the app crash. A logging library making the app crash is ironical... The project also uses specific version of okhttp that can't be changed now. How would I replace Sentry okhttp implementation when using the Sentry AGP, and Sentry BOM?

Thanks

SerggioZS avatar Feb 27 '24 15:02 SerggioZS

Let's:

  • Update the table in the README
  • Rename table header title "Android API" to "Minimum Android API"
  • Compatibility table with Sentry-Self Hosted / min SDK version
  • Have an overview with major SDK level requires which min SDK version

markushi avatar Feb 28 '24 14:02 markushi

Let's consider adding the min-api level to the release registry (requires craft and docs update)

markushi avatar Feb 28 '24 14:02 markushi