APIレスポンスエラー時の表示処理

domi10momo commented 3 years ago

closes #41

目的・概要

APIレスポンスエラー時、システムエラーが表示されることを改善

実装内容

レスポンスのHTTPステータスコードによって、成功か失敗かを判断
失敗の場合、期間指定画面にてレスポンスエラーの内容を表示

UI変更概要

エラー内容を期間指定画面に表示

画面キャプチャ

稼働確認チェック

[x] ローカル環境での動作確認
[x] rspecの実行＆エラーなし
[x] rubocopの実行＆コード修正
稼働確認

ベアラートークン不正時　 401内容表示
APIエンドポイント不正時　404内容表示
指定日付不正時　　　　　 422内容表示

参考資料

連絡事項

great084 commented 3 years ago

確認しました。ちょうど私はlimit overだったので429のエラーも確認できています。 APIから返ってきたエラー処理のハンドリングはできていると思います。一旦、この内容でマージしてしまって構いませんが、以下の内容を別途Issueにあげて、改善してほしいです。

ただ、ユーザに伝える（画面で表示する）エラーメッセージは再考の余地があります。端的に言うとAPIのエラー内容を、ユーザ向けに翻訳が必要です。

画面に表示するエラーメッセージは、エラーの状況をユーザが分かるように簡潔に伝え、ユーザのアクションに繋げる必要があります。例えば、以下のようなメッセージが画面に表示された際に、ユーザはどんなアクションがとれるでしょうか？ ※いずれもアプリ開発側の設定不備などのバグなので何もできないはず。

        { code: "403", message: "このリソースへのアクセスは許可されておりません。" },
        { code: "404", message: "リソースが見つかりませんでした。URLが間違っている可能性があります。" },
        { code: "422", message: "入力値が無効です。" },

domi10momo commented 3 years ago

@nozomi @great084 ひろみち

ありがとうございます。一旦マージし、指摘事項は別イシュに作成いたします。

great084 / twitter_tool