Use method instead of getters

[aloisdeniel]

I would preferably used methods for instead of getters that returns a new Future.

I find it more clear that an action is triggered by calling a method in a fire and forget case. At first I thought that there were a cache of the Future.

AppReview.storeListing; // I would find `AppReview.storeListing()` more transparent on the fact that a processing is potentially triggered.

[rodydavis]

Yes i will refactor on a future update!

[shinsenter]

@aloisdeniel
This may be resolved with No #38.

[tzachov]

As far as I see this issue was not resolved.
I agree with @aloisdeniel - getters for Futures feels unnatural.

I think all of these getters should be methods:

AppReview.isRequestReviewAvailable
AppReview.requestReview
AppReview.writeReview
AppReview.storeListing
AppReview.getAppID
AppReview.getiOSAppID

[rodydavis]

I'm open for PRs!