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

Initial .net 8/9 converstion #57

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Initial .net 8/9 converstion #57

wants to merge 3 commits into from

Conversation

jamesmontemagno
Copy link
Owner

Initial .net 8/9 converstion

PR Classification

Code cleanup and modernization of project structure.

PR Summary

This PR simplifies code, updates project configurations, and modernizes platform-specific implementations.

  • CrossStoreReview.shared.cs: Simplified Current property getter and updated platform-specific compilation symbols.
  • StoreReview.Plugin.csproj: Changed project SDK, updated target frameworks, and added new properties for packaging and platform support.
  • StoreReviewImplementation.android.cs: Replaced Xamarin dependencies with Microsoft.Maui and updated OnComplete method.
  • StoreReviewImplementation.apple.cs: Added a check for iOS 16.0 or higher.
  • StoreReview.sln: Updated Visual Studio version and removed several test projects and their configurations.

Make store review methods async and return Task<bool>

Updated IStoreReview interface and implementations across Android, iOS, macOS, tvOS, and UWP to make OpenStoreListing and OpenStoreReviewPage methods asynchronous, returning Task<bool> instead of void.

- Changed method signatures in IStoreReview.shared.cs.
- Updated Android implementation to return Task.FromResult(true/false).
- Modified Apple implementation to use UIApplication.SharedApplication.OpenUrlAsync.
- Updated UWP implementation to handle exceptions and return Task<bool>.
- Removed PlatformExtensions class from UWP implementation.
Updated README.md to reflect .NET 8+ support for iOS, Android, MacCatalyst, and Windows, and removed outdated platform information. Changed UWP method signatures to return `Task<bool>` for `OpenStoreListing` and `OpenStoreReviewPage`. Updated Android setup instructions in README.md to remove outdated steps. Refactored `CrossStoreReview.shared.cs` and `StoreReviewImplementation` files for modern C# syntax and improved nullability. Simplified `IStoreReview.shared.cs` interface. Updated `StoreReview.Plugin.csproj` with correct metadata. Improved exception handling and logging across all platforms.
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.

1 participant