NativeScript / NativeScript/plugins
[@nativescript/biometrics] Android `promptForPin` method checks options for `pinFallBack` instead of `pinFallback`
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- TypeScript
- Sterne
- 206
- Forks
- 123
- Ø Merge
- 2 T. 1 Std.
- Gemergte PRs (30 T.)
- 1
Beschreibung
While debugging, we found that in the promptForPin() method in the android code for biometrics, it's checking for the wrong property.
if (!options?.pinFallBack) {
should be
if (!options?.pinFallback) {
NOTE: We can fix it by passing both pinFallBack and pinFallback through on the options, but it would be best if this were fixed.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginnen Sie bei packages/biometrics/index.android.ts ungefähr in Zeile 426 und prüfen Sie die Optionenprüfung von promptForPin(). Korrigieren Sie den inkonsistenten Namen der Fallback-Eigenschaft und verifizieren Sie anschließend, dass promptForPin() die dokumentierte Option pinFallback erkennt, ohne die falsch geschriebene Variante zu erfordern.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- android, typescript
- Bereich
- authentication, mobile
- Issue-Typ
- Bug
- Schwierigkeit
- 1/5
- Geschätzter Aufwand
- Unter einer Stunde
- Aktivitätsstatus
- Veraltet
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 72/100