pj-picbook / picbook

9 stars 0 forks source link

Crashlyticsによるエラー対応 #126

Closed Ta23ka98 closed 2 years ago

Ta23ka98 commented 2 years ago

概要

要件

Ta23ka98 commented 2 years ago

@karokita Crashlyricsの設定をしたいのですが、リポジトリ名がどっちか忘れてしまいました😅 どちらが開発用のリポジトリだったか教えて頂きたいです🙇

スクリーンショット 2022-08-31 19 53 49
karokita commented 2 years ago

@Ta23ka98 紛らわしいですね😂 すみません!

dev:pj-picbook-pro pro:pj-picbook

になります。

Ta23ka98 commented 2 years ago

@karokita 了解です! ありがとうございます🙌

Ta23ka98 commented 2 years ago

@karokita 改めてこのissueの要件を詰めたいです🙋 とりあえず現在、アプリが問答無用でクラッシュした等のレポートは送られるみたいです。

他にCrashlyticsの機能として、"致命的でないエラー"を送信するように実装することも可能です。 (例えばログイン・会員登録で「ユーザが見つかりません」などの小さいエラー)

もしこれらに対応する場合、「どういうエラーに対応するか」を決める必要が出てくると思います🤔

karokita commented 2 years ago

@Ta23ka98

おお、クラッシュした際のレポート送られるようになったんですね。ありがとうございます!

一旦、初期はユーザー数などを考えていくと クラッシュした内容を送れるというとこまでissue closeが良いかなと思いました!

ちなみに見方としては、コンソール→crashlyticsで見れる感じでしょうか?

Ta23ka98 commented 2 years ago

@karokita 一旦closeは了解しました!🙌 Androidでは確認できていないので、エミュレータでも確認次第プルリク出そうと思います。

おっしゃる通り、Crashlyticsはこんな感じで見れます👍

スクリーンショット 2022-09-03 20 43 57
karokita commented 2 years ago

@Ta23ka98 おおお、見れました!!! 見てる感じだとどのファイルのどの行でクラッシュしてるかがとれるんですかね...🤔

データ量が多くなったり、ユーザー数が多くなるとクラッシュデータ取れるの大事になってくると思うので、 まずはCrashlytics入れられてよかったなと思いました!

Androidでのご確認、よろしくお願いします🙋‍♀️

Ta23ka98 commented 2 years ago

@karokita Androidでのクラッシュも確認できたので、プルリク作成しました!

未解決の課題(致命的でないエラーへの対応)があるので、一応チェックボックスを空欄にしてあります。 「現時点でissue作成は不要だからチェック」/あるいは「後で作成するので未チェック」の判断はお任せします🙇

クラッシュの種類については、上から順に ①Flutter:たぶん▶️ボタンで再起動した報告 ②first_page, line??:FirstPageでテストクラッシュを確認した時の報告と、その行数 ③signup_notifier, line:お試しで"致命的でないエラー"(会員登録できなかったエラー)を確認した時の報告 という感じだと思います👍

ドキュメント曰く、カスタムキーを設定すればどのようなエラーかが簡単に可視化できるようです (あと③についてはお試しなので、コミットせずプルリクにも反映されていない感じです)

karokita commented 2 years ago

@Ta23ka98

一旦issue立ててみましたので、チェックボックスお願いします🙋‍♀️ また、丁寧に説明いただいてありがとうございます〜!

crashlyticsのダッシュボードの下の画面の問題のところで、新規/解決済みみたいな項目があったので、 何かエラーをキャッチして、それを解消する運用までをcrashlyticsでやっていくイメージなんですかね...🤔

キャッチしたいエラーとどう使っていくかを想像しつつ、パターン出しできると良さそうですね!

Ta23ka98 commented 2 years ago

こちらマージ済みなのでクローズしておきます🙋