NativeScript / NativeScript/plugins

[@nativescript/biometrics] Android `promptForPin` method checks options for `pinFallBack` instead of `pinFallback`

Ouverte Adaptée aux débutants
#640 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Langage dominant
TypeScript
Étoiles
206
Forks
123
Merge moyen
2 j 1 h
PR mergées (30 j)
1

Description

While debugging, we found that in the promptForPin() method in the android code for biometrics, it's checking for the wrong property.

https://github.com/NativeScript/plugins/blob/a503c40264e6704cbf87ccaaa76da5b8486b4e46/packages/biometrics/index.android.ts#L426

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.

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez dans packages/biometrics/index.android.ts, vers la ligne 426, et examinez la vérification des options de promptForPin(). Corrigez le nom incohérent de la propriété de fallback, puis vérifiez que promptForPin() reconnaît l’option documentée pinFallback sans nécessiter la variante mal orthographiée.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
android, typescript
Domaine
authentication, mobile
Type d'issue
Bug
Difficulté
1/5
Temps estimé
Moins d'une heure
Activité
À l'abandon
Clarté
Clairement spécifiée
Accessibilité débutants
72/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.