yumemi-inc / flutter-mobile-project-template

MIT License
36 stars 7 forks source link

Refactor AppStatus provider to use existing state instances directly #246

Closed daichikuwa0618 closed 5 months ago

daichikuwa0618 commented 5 months ago

概要

245 の作り直し。

appStatus provider において、 MaintenanceModeStatusForceUpdateStatus の不要なインスタンス再生成があったので修正しました。

問題点

インスタンス生成のオーバーヘッドやメモリ使用量の増大はわずかだと思いますが、都度生成していると今後 MaintenanceModeStatusForceUpdateStatus に、メンバーの追加があるがコンストラクタでデフォルト値が設定されているという場合に古い値が無視されてしまうことが懸念されます。

レビュー観点

レビューレベル

レビュー優先度

画像 / 動画

この変更後にメンテナンスモードと強制アップデートの状態変更が監視できているのは確認しました。

github-actions[bot] commented 5 months ago

Ready for review :rocket:

github-actions[bot] commented 5 months ago

Visit the preview URL for this PR (updated for commit 1b30ebd):

https://flutter-mobile-project-template-catalog--pr246-fix-unn-f6idrkw7.web.app

(expires Tue, 18 Jun 2024 00:57:33 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 9ea56735a63d07a7cfe62eb204b0528284c37c23