search

aboutyou / dart_packages

Dart and Flutter plugins maintained and used by @ABOUTYOU
222 stars 152 forks source link

I hope to add an option to remove Android dependencies, as Apple login is generally not needed on Android devices. #409

Closed winter-tech closed 8 months ago

winter-tech commented 8 months ago

I hope to add an option to remove Android dependencies, as Apple login is generally not needed on Android devices.

If it doesn't work, I hope to upgrade the version of Kotlin on the Android side to 1.6.20 or higher. It is now 1.6.0, and some properties have a minimum of 1.6.20, so I cannot run it on the Android side.

tp commented 8 months ago

I don't think there is a good way to remove / split the Android part technically, and generally I think users of this package offer Sign in with Apple on Android (and other platforms) if they have it on iOS in order to allow users to migrate systems and access their data on any device.

But we could see that we upgrade the Android dependencies again to align with the latest best practices.

winter-tech commented 8 months ago

I don't think there is a good way to remove / split the Android part technically, and generally I think users of this package offer Sign in with Apple on Android (and other platforms) if they have it on iOS in order to allow users to migrate systems and access their data on any device.

But we could see that we upgrade the Android dependencies again to align with the latest best practices.

When will it be updated?

tp commented 8 months ago

@winter-tech Should be a quick fix via #411.

In the meantime you can use the preview release 6.1.0-dev.1 from https://pub.dev/packages/sign_in_with_apple/versions/6.1.0-dev.1.

Please let me know if that works for you :)

winter-tech commented 8 months ago

@winter-tech Should be a quick fix via #411.

In the meantime you can use the preview release 6.1.0-dev.1 from https://pub.dev/packages/sign_in_with_apple/versions/6.1.0-dev.1.

Please let me know if that works for you :)

Thank you, it's ready to run now