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

Migrate to null-safety #24

Closed
wants to merge 5 commits into from
Closed

Migrate to null-safety #24

wants to merge 5 commits into from

Conversation

LeonardoRosaa
Copy link

@LeonardoRosaa LeonardoRosaa commented Jul 18, 2021

Starting pull request to migrate package to null safety version.

To do:

  • Migrate to null safety;
  • Test in IOS plataform;
  • Test in Android plataform.

sagargpt and others added 3 commits September 3, 2021 14:45
* Building a flutter android apk failed due to the android project being androidX compatible. Hence, updating the compileSdkVersion to 27 to enable androidX migration.
* In case of result.error from AppAvailability.java the result received on dart is not of type Map<dynamic, dynamic> instead it's a Platform Exception. Since, the type cast is explicitly on the invokeMethod call, this results in type cast exception.

* Resolving this by storing the result in an immutable variable first and then typecasting.
Fix: Resolving Type Cast Error
@GRBurst
Copy link

GRBurst commented Sep 7, 2021

@pichillilorenzo any updates on this? Is this library still maintained?

@sagargpt
Copy link

sagargpt commented Sep 8, 2021

Is anyone actively reviewing the PR?
There seems to be no assigne or reviewers attached.

@juliavi
Copy link

juliavi commented Sep 15, 2021

Is anyone reviewing this and going to merge? @pichillilorenzo

@juliavi
Copy link

juliavi commented Sep 24, 2021

@pichillilorenzo

@Nei1eveN
Copy link

Nei1eveN commented Jan 24, 2022

any update on this?

@LeonardoRosaa
Copy link
Author

LeonardoRosaa commented Feb 4, 2022

Hey folks, I writing a similar package, from here after some weeks I launch first release for Android only. I doing make a skeleton first to launch and open for public.

But I warning, I need help to write for IOS platform.

Thanks!

@LeonardoRosaa
Copy link
Author

@Yash-Garg
Copy link

Yash-Garg commented Mar 17, 2022

any update on this?

Hey, I recently refactored this whole package in Kotlin. Here's the link if you want to use it - https://github.com/Yash-Garg/AppCheck

Works for both Android and iOS, including null safety.

@rahuldange09
Copy link

Great! Thanks.
Until @pichillilorenzo merges this PR.
Use it like this

  flutter_appavailability:
    git: https://github.com/LeonardoRosaa/flutter_appavailability

@LeonardoRosaa LeonardoRosaa closed this by deleting the head repository Nov 11, 2022
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.

7 participants