患者作成時、存在しない電話番号などで、SMS送信に失敗した場合でも登録自体はできてしまう

[halsk]

患者作成時にSMS送信に失敗した場合、本来Patientは作成されないべきだが、実際には登録できてしまう。

https://github.com/codeforjapan/remote-patient-monitoring-api/blob/6ce0bddd506ce5634f4ba5859a8dfe1f63d96281/src/lambda/patient.ts#L260-L269

でエラーコードを一応返している。（実際には #156 と相まってCORSエラーになるが）

SMSで送るログインURLを作るには一度レコードを作らないと行けないので上記のようになっているが、本来であれば、エラーになった時点で作成したレコードを削除すべきである。

[kenichisak]

SMS を送信しない場合は、不正な電話番号であっても 検知・削除できないので このタイミングで Patient のレコードは削除しなくてもいいのではと個人的に思いました。

患者を削除する機能もあったと思いますし、別の機能として、電話番号を変更したり SMS を再送する機能がもしあれば問題ない気がします。