KasemJaffer / receive_sharing_intent

A Flutter plugin that enables flutter apps to receive sharing photos, text and url from other apps.
Apache License 2.0
325 stars 375 forks source link

Need Library Maintenance Help? #182

Open GRBurst opened 3 years ago

GRBurst commented 3 years ago

Hi 👋 ,

first of all, thank you for creating this library. It's a very useful and fundamental part of many flutter Apps.

Unfortunately there are many outstanding important Pull Requests for this library, that never got merged. It looks like, you could need some help with maintenance. I suggest to give some contributors more permissions, so they can help reviewing and merging pull requests.

I suggest giving maintenance permission to some of your contributors like @danReynolds, @kr1tzy and @czernas as well as people with open pull requests, e.g. @xal, @Klabauterman, @MagTuxGit, @thegrxp or @odam1

Happy to hear from you @KasemJaffer

dvird commented 2 years ago

@GRBurst please open new fork and release new version name to pub with all the fixes here.

jigarfumakiya commented 2 years ago

Any update on this Guys. Did anyone fork this project?

MagTuxGit commented 2 years ago

Any update on this Guys. Did anyone fork this project?

I did. But didn't publish as a package. https://github.com/MagTuxGit/receive_sharing_intent Recently updated and testing on my projects.

Using the next way:

  receive_sharing_intent:
    git:
      url: https://github.com/MagTuxGit/receive_sharing_intent
      ref: 3692dcac0d1c3c70175fdad8304936bc89d2a99a

All functionality the same as in this package, but added urls stream.

  void _observeUniLinks() async {
    try {
      final String? initialUri = await ReceiveSharingIntent.getInitialLink();
      _handleUniLink(initialUri);
    } on PlatformException catch (e) {
      _handleUniLinkError(e);
    }

    _subscriptionForLink =
        ReceiveSharingIntent.getLinkStream().listen((String link) {
      _handleUniLink(link);
    }, onError: (err) {
      _handleUniLinkError(err);
    });
  }
jigarfumakiya commented 2 years ago

Thanks @MagTuxGit

sebastianhaberey commented 2 years ago

It's crazy to think there's 141 forks (142 now including mine) and that most of those developers probably fixed the same issues over and over again 🤪 But it's understandable too, many maintainers are extremely busy in their daytime jobs and even though their intentions are noble at the start, they find they don't have the time to keep up.

Anyways I like @GRBurst 's idea of having multiple maintainers / collaborators.

P.S.: @MagTuxGit sounds good I'll give it a try as well!

KasemJaffer commented 2 years ago

Thank you all for the support and I apologize for the late response. Thats because i am indeed extremely busy. Would like to have some help from the community to maintain the plugin. I added @GRBurst as a contributor because i think he/she has been helpful reviewing other PRs. If anyone is interested please let me know here.

KasemJaffer commented 2 years ago

I also added @danReynolds, @kr1tzy and @czernas

MagTuxGit commented 2 years ago

Nice. I'll rollback to the published version then and try to make a PR with my changes (without links handling, decided to use uni_links instead).