Modernize all the things

avioli / uni_links

Flutter plugin for accepting incoming links.

BSD 2-Clause "Simplified" License

563 stars 303 forks source link

Modernize all the things #92

Closed hacker1024 closed 3 years ago

hacker1024 commented 3 years ago

This PR contains a slew of changes:

Migrate to null safety (resolves #90)
Migrate to a federated plugin architecture (paves the way for #51)
Migrate to v2 Android embedding APIs (resolves #85)
Use more effective Dart guidelines in the example

As a part of this, it's worth noting that I removed the exception documentation in a few functions. The exceptions documented should never be thrown by this plugin unless there's a bug; developers should not have to handle them.

hacker1024 commented 3 years ago

This PR can supersede #69 because:

It supports null safety.
Platform-independent code like the URI stream is in the right spot - in #69, it's in the platform interface instead of the main plugin, which is incorrect.

miDeb commented 3 years ago

@avioli may I ask if you plan on merging this pr soon? FYI, if you don't want to mantain this package anymore you can also transfer ownership to fluttercommunity.

avioli commented 3 years ago

Thank you @hacker1024

espresso3389 commented 3 years ago

Could you please tell me when do you release the new plugin version that contains the PR?

IchordeDionysos commented 3 years ago

@avioli there is no new release yet with the null-safety features ☺️

Could you publish a release to pub.dev? Would be really awesome 😍