UnUniFi / utils

10 stars 2 forks source link

ユーザーフレンドリーなエラーメッセージの実装 #200

Closed YasunoriMATSUOKA closed 2 years ago

YasunoriMATSUOKA commented 2 years ago

Is your feature request related to a problem? Please describe. A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

CDPに関して以下のような原因でトランザクションを実行できない場合があるのですが、全てStatus Code 400 or 500的なエラーになってしまい、ユーザーにとって何が問題なのか伝えることができていない現状があります。

Describe the solution you'd like A clear and concise description of what you want to happen.

フォームやサービス等で適切にバリデーションやエラー通知を行えるように修正したいという内容になります。(CDPだけでなくAuctionについても本質的には同様の問題がありそうですが、現時点ではAirDropタスクにはなっていないので、優先度は若干低いと思っています。)

Describe alternatives you've considered A clear and concise description of any alternative solutions or features you've considered.

Additional context Add any other context or screenshots about the feature request here.

kimurayu45z commented 2 years ago

秘密鍵の入力ミスに関しては、入力された秘密鍵から公開鍵を導出して、その公開鍵がindexedDBで管理しているそれと一致するかバリデーションする手続きを挟むことで、トランザクション送るまでもなく対処できそうだなと思いました。

taiki1frsh commented 2 years ago

このIssueに関する進捗状況を知ることはできるでしょうか。

taro04 commented 2 years ago

@taiki-furu

おつかれさまです。 (このIssueの認識はなかったですが、)以下PRで現在進めています。 https://github.com/UnUniFi/utils/pull/207

・秘密鍵の入力ミス ⇒ スナックバーで通知。 ・担保率不足状態になるトランザクションの試行 ⇒ CDPからデータ持ってきてバリデーション ・残高不足 ⇒ 所持数でバリデーション。 ・手数料不足 ⇒ スナックバーで通知。

YasunoriMATSUOKA commented 2 years ago

Completed