NativeScript / NativeScript/plugins

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

Aberta Para iniciantes
#640 0 comentários 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Linguagem predominante
TypeScript
Estrelas
206
Forks
123
Merge médio
2d 1h
PRs com merge (30d)
1

Descrição

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.

Guia de contribuição

Abrir o guia de contribuição

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Direção de pesquisa

Comece em packages/biometrics/index.android.ts, por volta da linha 426, e inspecione a verificação das opções de promptForPin(). Corrija o nome inconsistente da propriedade de fallback e, em seguida, verifique se promptForPin() reconhece a opção documentada pinFallback sem exigir a variante com erro de grafia.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Stack de tecnologia
android, typescript
Domínio
authentication, mobile
Tipo de issue
Bug
Dificuldade
1/5
Tempo estimado
Menos de uma hora
Status de atividade
Estagnada
Clareza
Claramente especificada
Facilidade para iniciantes
72/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.