(優先度：中)(Critical?：Middle)処理番号の入力欄について、数字以外の入力を無効にする - Githubissues

Covid-19Radar / Covid19Radar

Open Source / i18n / iOS Android Cross Platform Contact Tracing App by exposure notification framework Xamarin App and Server Side Code

https://github.com/Covid-19Radar/Covid19Radar/blob/master/CONTRIBUTING.md

Mozilla Public License 2.0

47 stars 9 forks source link

(優先度：中)(Critical?：Middle)処理番号の入力欄について、数字以外の入力を無効にする #706

Closed Niccari closed 4 years ago

Niccari commented 4 years ago

Purpose

"陽性情報の登録"について、処理番号の入力欄に数字以外も入力できるようになっていました※ "登録する"ボタンをタップしたときに数字のみかどうかバリデーションされますが、番号入力時に制限しておくことで番号のみ入力できることを直感的にわかるようにしています。

※ 数字キーボードで入力する設定はされていましたが、入力直後の制限はありませんでした。また、"."が入力可能になっていました。

Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type

What kind of change does this Pull Request introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

Get the code

git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install

Test the code
数字1桁を入力すると、そのまま入力欄に反映される
数字以外を入力しても、入力欄は更新されずそのままになる

What to Check

Verify that the following are valid

入力欄には数字のみ入力できること
入力欄に数字以外を入力したとき、入力欄が更新されないこと
"登録する"をタップし入力内容をバリデーションするときに、入力欄の入力内容と_diagnosisUidの値が一致していること

Other Information

kazumihirose commented 4 years ago

まず、PRありがとうございます。こちらは厚生労働省によって運用されているCOCOAのコードではありません。大変恐縮なのですが、厚生労働省(COCOA)に再度Issue/PRなど頂ければ幸いです。何卒宜しくお願い致します。ただ、非常にありがたいご提案でして、Covid19Radarでもマージさせていただきました。