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

Add iOS Widget #666

Merged
merged 91 commits into from
Aug 24, 2022
Merged

Add iOS Widget #666

merged 91 commits into from
Aug 24, 2022

Conversation

bannzai
Copy link
Owner

@bannzai bannzai commented Aug 20, 2022

Abstract

Why

Links

Memo

色々と引っかかって今のところ実機でしか試せない

SendProcessControlEvent:toPid: encountered an error: Error Domain=com.apple.dt.deviceproce…

  • M1 Mac SimulatorだとEXCLUDE_ARCHSにarm64を指定しないとWidgetが起動しないらしい
  • arm64をExcludeするとGoogle SignInがarm64を除外しない限りビルドができないorz。6系では直っているらしいが、生憎Flutterのパッケージのバージョンは5系統の模様
    • ref: SendProcessControlEvent:toPid: encountered an error: Error Domain=com.apple.dt.deviceproce…
  • 実機ではENABLE_BITCODEをオフにするとビルドはできた、が動作確認はまだできていない(WIP)

Checked

  • Analyticsのログを入れたか
  • Navigator.of(context).pop() の後にContextを使用したメソッドを実行していない
  • 境界値に対してのUnitTestを書いた
  • パターン分岐が発生するWidgetに対してWidgetTestを書いた
  • リリースノートを追加した

@bannzai
Copy link
Owner Author

bannzai commented Aug 24, 2022

以下の理由によりissueをクローズ

  • Widgetは服用完了みたいなリアルタイムの状態を反映するのは保障できない
    • とはいえreloadTimelinesをすれば更新はされる場合が多いが、多少ラグがある場合があるのがネック
    • ちなみに時計やカレンダーのような現在時刻・もしくは現在時刻との差分をテキストとして表示する機能はあるのでカレンダー表示には向いてそう
  • 未来の日程を決める機能みたいなリアルタイム性があまり求められないが定期的に変わるくらいの頻度のデータであれば整合性は保てそう
  • もしくはユーザーのアクション関係なしに 今日の服用番号 のみを表示する。等の状態であれば表示しても問題なさそう

@bannzai bannzai marked this pull request as ready for review August 24, 2022 12:18
@bannzai bannzai merged commit ef9f74e into main Aug 24, 2022
@bannzai bannzai deleted the add/widget/first branch August 24, 2022 12:18
This was referenced Aug 24, 2022
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