flutter_compass icon indicating copy to clipboard operation
flutter_compass copied to clipboard

Revert minSdkVersion back to 20

Open TheoLassonder opened this issue 4 years ago • 10 comments

Recently the minSdkVersion was changed from 20 to 23 but there doesn't seem to be any reason for that. We are still supporting devices on API levels lower than 23, so can we please keep doing so?

TheoLassonder avatar Sep 20 '21 00:09 TheoLassonder

I also used SDK 19 before. Unfortunately now must be at least v23.

aqinfh avatar Sep 21 '21 10:09 aqinfh

@aqinfh I must be missing something - why must it be at least v23? When I set the minSdkVersion back to 20 it seems to work fine.

TheoLassonder avatar Sep 21 '21 10:09 TheoLassonder

have you tried to make a release version? idk flutter tells me to update minsdk version, or downgrade flutter_compass version

aqinfh avatar Sep 29 '21 23:09 aqinfh

@aqinfh Sorry I meant if I clone this repo and set the minSdkVersion back to 20 it seems to work fine on the device. I've created a pull request to change it back.

TheoLassonder avatar Oct 02 '21 04:10 TheoLassonder

what version of flutter are you using? mine v2.5.0

aqinfh avatar Oct 02 '21 04:10 aqinfh

Flutter 2.5.1

TheoLassonder avatar Oct 02 '21 05:10 TheoLassonder

i'm using flutter with the same version now.

When i use Flutter compass 0.7.0. still need to update min sdkversion

image

aqinfh avatar Oct 04 '21 09:10 aqinfh

@aqinfh I've cloned the compass repo and changed it to set the minSdkVersion back to 20 and then built my Flutter project with the cloned repo. There is special notation in pubspec.yml to refer to a (cloned) git (Github) repo.

TheoLassonder avatar Oct 04 '21 22:10 TheoLassonder

what did you mean about "There is special notation in pubspec.yaml to refer to a (cloned) git (Github) repo"?

aqinfh avatar Oct 05 '21 05:10 aqinfh

@aqinfh you can use:

flutter_compass: #^0.7.0 # Here the git ref is used because current 0.7.0 forces android min sdk to be 23
    git:
      url: https://github.com/hemanthrajv/flutter_compass.git
      ref: 14e134f0aed3c9fe09065bea05281f2c26f171d1

That is the special notation mentioned above, in this PR 67 the minSdkVersion is changed to 20. That PR is already merged so it should be available in the next version after 0.7.0. In the mean time you can use the git notation referencing the specific commit with the PR fix.

luis901101 avatar Oct 15 '21 21:10 luis901101