fluttercommunity / rx_command

RxCommand - Reactive event handler wrapper class inspired by ReactiveUI. Maintainer @escamoteur
https://pub.dev/packages/rx_command
MIT License
134 stars 19 forks source link

Improve Readme and doc #54

Open Mabsten opened 3 years ago

Mabsten commented 3 years ago

I really appreciate this package but I find the documentation difficult for newcomers:

I am inexperienced and unfamiliar with git but am available to help update the documentation. Personally, the guide that got me aficionado of this package was the blog article https://www.burkharts.net/apps/blog/rxvms-foundations-rxcommand-and-getit/, together with the two articles of the same series preceding it (which is much more complete than "Making Flutter more Reactive" mentioned in the Readme). One idea could be to update this series of articles to the current RxCommand and RxDart Api and make it available in the wiki of this repo.

escamoteur commented 3 years ago

Hi, I completely understand your frustration. It's just a matter of time and energy from my side. Additionally I personally moved to use flutter_command which does no longer rely on streams together with my functional_listener package which offers rx like methods for ValueListenables.

It would be awesome if you could help updating the readme here. If you need help with git, no problem, I'm happy to help.

Basically you do a fork (button on the main page of the repo) to your own github account. Then you clone that repository to your machine and make the changes. After that you commit and push the changes back to your copy of this repository. After that you can trigger a PR (pull request) on the github page of your forked repository which allows then me to review and merge your changes.

Mabsten commented 3 years ago

Thank you for your answer. I am happy to contribute. Within half a day I will update the readme.

Mabsten commented 3 years ago

I have updated the README with https://github.com/fluttercommunity/rx_command/pull/55 I remain available if needed to update the Doc Api (which is much more obsolete than the Readme) and to write some Doc for newcomers (the various parameters emitInitialCommandResult, etc, are not easy to understand even from the doc api)

escamoteur commented 3 years ago

Sorry for the delay just merged and pushed with the V6.0.0 many thanks!!! I would be really happy if you help to improve the docs!