smstuebe / xamarin-fingerprint

Xamarin and MvvMCross plugin for authenticate a user via fingerprint sensor
Microsoft Public License
494 stars 118 forks source link

CA1416 analyzer errors (This call site is reachable on...) for .NET MAUI version #222

Open jfversluis opened 2 years ago

jfversluis commented 2 years ago

When installing the 3.0.0-beta1 version on a .NET MAUI project I get CA1416 warnings all over the place. We had that happening as well, so you probably want to have a look at the fixes we did. See: https://github.com/dotnet/maui/issues/823

The app runs regardless, but the warnings might be confusing. Only Android is shown in the screenshot below, but happens on all platforms.

image

Steps to reproduce

  1. Create new .NET MAUI project
  2. Install Plugin.Fingerprint 3.0.0-beta1
  3. Start adding the Plugin.Fingerprint code

Expected behavior

No analyzer warnings

Actual behavior

Analyzer warnings all over the place to inform the user about supported platform versions

Crashlog

N/A

Configuration

Version of the Plugin: 3.0.0-beta1

Platform: Android, iOS, Mac Catalyst, Windows

Device: N/A

smsissuechecker commented 2 years ago

Hi @jfversluis,

I'm the friendly issue checker. Thanks for using the issue template :star2: I appreciate it very much. I'm sure, the maintainers of this repository will answer, soon.

ghost commented 2 years ago

Can anyone help with this issue?

buyaa-n commented 2 years ago

Hope this doc would help Handle reported warnings, let us know if you have any more questions

ghost commented 1 year ago

Hope this doc would help Handle reported warnings, let us know if you have any more questions

Hi @buyaa-n, The component is requesting at least Android 31.0 and as the Android 31.0 version is a recent version and will disqualify many devices. Will the minimum version for MAIU be Android 31.0?

buyaa-n commented 1 year ago

I am not familiar with this repo, so I am not sure if this should be fixed in this repo or MAUI, if this repo is not using anything that require Android 31.0+ this repo should update its minimum version down