Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ローカル通知再チャレンジ #869

Merged
merged 131 commits into from
Jul 24, 2023
Merged

Conversation

bannzai
Copy link
Owner

@bannzai bannzai commented Jul 16, 2023

Abstract

Why

Links

#558
#734
#866

Test

通知

各アクション後に未服用で通知が来る

これは全部テスト

  • ピルシート追加
  • 服用記録
  • 服用キャンセル
  • ピルをタップして服用
  • ピルをタップして服用キャンセル
  • クイックレコード
  • 服用時刻編集
  • 通知の文言を変えた時
  • 休薬期間の通知のON/OFF
  • 手動の休薬終了後
  • 初期設定完了後
  • サインイン
  • 番号変更後
  • リマインダーの通知がOFF->ONになった時
  • 久しぶりにアプリを開いたが、通知がスケジュールされていない時: ログで確認
  • トライアル終了後/プレミアム加入後 → これは服用は続けられているので何もしない。有料機能をしばらく使えてもヨシとする

各アクション後に服用済みにして通知が来ない

どれか数個で良い

  • ピルシート追加
  • 服用記録
  • 服用キャンセル
  • ピルをタップして服用
  • ピルをタップして服用キャンセル
  • クイックレコード
  • 服用時刻編集
  • 通知の文言を変えた時
  • 休薬期間の通知のON/OFF
  • 手動の休薬終了後
  • 初期設定完了後
  • サインイン
  • 番号変更後
  • リマインダーの通知がOFF->ONになった時
  • 久しぶりにアプリを開いたが、通知がスケジュールされていない時
  • トライアル終了後/プレミアム加入後 → これは服用は続けられているので何もしない。有料機能をしばらく使えてもヨシとする

各アクションを服用済みの状態にして何もしない状態で通知が来ない

どれか数個で良い。できれば一つ上の 各アクション後に服用済みにして通知が来ない と被らない

  • ピルシート追加
  • 服用記録
  • 服用キャンセル
  • ピルをタップして服用
  • ピルをタップして服用キャンセル
  • クイックレコード
  • 服用時刻編集
  • 通知の文言を変えた時
  • 休薬期間の通知のON/OFF
  • 手動の休薬終了後
  • 初期設定完了後
  • サインイン
  • 番号変更後
  • リマインダーの通知がOFF->ONになった時
  • 久しぶりにアプリを開いたが、通知がスケジュールされていない時
  • トライアル終了後/プレミアム加入後 → これは服用は続けられているので何もしない。有料機能をしばらく使えてもヨシとする

表示

  • Premium時は番号と日付が出る
  • Trial時は番号と日付が出る
  • ピルシートの表示モードごとに表示が変わる(番 or 日)
    • これは最悪やらなくて良い
  • DisplayNumberSettingを変更した際の通知の表示

条件

  • 手動の休薬期間は通知が来ない
  • 設定から通知をOFFにしているときは通知が来ない
  • 設定から通知をONにした時に通知が来る
  • 設定からの休薬期間の通知がOFFの場合は通知が来ない
  • 破棄したら通知時刻になっても通知が来ない
  • useLocalNotificationがfalseの場合はリモート通知

境界値

  • 番号表示でピルシートとピルシートの境目の番号表示: ログで確認
  • 新しいシート作成がOFFの場合のピルシートグループが終わりの場合に通知が来なくなる: ログ見ながら確認
  • 新しいシート作成がONの場合の先読み登録: ログ見ながら確認
  • 未服用の状態で日付を跨いでも通知が来る

クイックレコード

  • アプリのプロセスがバックグラウンドにある時にクイックレコードをすると服用済みになる
  • アプリのプロセスがキルされている時にクイックレコードをすると服用済みになる

Checked

  • Analyticsのログを入れたか
  • 境界値に対してのUnitTestを書いた
  • パターン分岐が発生するWidgetに対してWidgetTestを書いた

@bannzai bannzai merged commit 2a01246 into main Jul 24, 2023
4 checks passed
@bannzai bannzai deleted the imp/notification/local_notification branch July 24, 2023 01:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant