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

Introduce ReviewStatus enum and update RequestReview method #50

Merged
merged 5 commits into from
Mar 9, 2024

Conversation

jamesmontemagno
Copy link
Owner

@jamesmontemagno jamesmontemagno commented Feb 26, 2024

Fixes #49
A new enumeration ReviewStatus has been introduced in the Plugin.StoreReview namespace, including values Succeeded, Error, CanceledByUser, NetworkError, and Unknown. The RequestReview(bool testMode) method in multiple files has been modified to return a Task<ReviewStatus> instead of a Task. This method has been updated in StoreReviewImplementation.android.cs to return either ReviewStatus.Succeeded or ReviewStatus.Error based on the status variable, in StoreReviewImplementation.apple.cs to return ReviewStatus.Unknown, and in StoreReviewImplementation.uwp.cs to use the StoreContext.GetDefault().RequestRateAndReviewAppAsync() method and return a ReviewStatus based on the result. Additional using directives have been added to the StoreReviewImplementation.uwp.cs file.###

A new enumeration `ReviewStatus` has been introduced in the `Plugin.StoreReview` namespace, including values `Succeeded`, `Error`, `CanceledByUser`, `NetworkError`, and `Unknown`. The `RequestReview(bool testMode)` method in multiple files has been modified to return a `Task<ReviewStatus>` instead of a `Task`. This method has been updated in `StoreReviewImplementation.android.cs` to return either `ReviewStatus.Succeeded` or `ReviewStatus.Error` based on the `status` variable, in `StoreReviewImplementation.apple.cs` to return `ReviewStatus.Unknown`, and in `StoreReviewImplementation.uwp.cs` to use the `StoreContext.GetDefault().RequestRateAndReviewAppAsync()` method and return a `ReviewStatus` based on the result. Additional `using` directives have been added to the `StoreReviewImplementation.uwp.cs` file.
@jamesmontemagno jamesmontemagno merged commit ae458a6 into master Mar 9, 2024
1 check passed
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.

User not prompted for review on MAUI WinUI app
1 participant