NativeScript / NativeScript/plugins

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

未关闭 适合新手
#640 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

还没有人认领这个 Issue。

主要语言
TypeScript
星标
206
派生
123
平均合并
2 天 1 小时
30 天内合并 PR
1

描述

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.

贡献指南

打开贡献指南

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

调研方向

从 packages/biometrics/index.android.ts 第426行附近开始,检查 promptForPin() 的选项检查。修正不一致的 fallback 属性名称,然后验证 promptForPin() 能识别文档中说明的 pinFallback 选项,而不要求使用拼写错误的变体。

由索引模型根据 Issue 内容生成。

评估

技术栈
android, typescript
领域
authentication, mobile
Issue 类型
缺陷
难度
1/5
预计耗时
1 小时以内
活跃度
停滞
描述清晰度
描述清楚
新手友好度
72/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。