[@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.

Évaluation

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

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.

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.

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

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.

Autres issues de NativeScript/plugins

Toutes les issues de NativeScript/plugins

Issues similaires

Plus d'issues TypeScript

Recevez les nouvelles issues par e-mail

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