refactor: 強制アップデート関連の実装を features/force_update パッケージに切り出す - Githubissues

yumemi-inc / flutter-mobile-project-template

MIT License

37 stars 7 forks source link

refactor: 強制アップデート関連の実装を features/force_update パッケージに切り出す #310

Closed tatsutakein closed 3 months ago

tatsutakein commented 3 months ago

概要

305

強制アップデート関連の実装を features/force_update パッケージに切り出します。今回は追加・変更のみに留め packages/cores/core/lib/src/app_status から強制アップデート関連の機能を削除することは別 PR で対応する予定です。

レビュー観点

packages/cores/core/lib/src/app_status から強制アップデート関連の機能を削除する必要がありますが、現時点で変更ファイル数が多くなっているため別 PR とさせてください

レビューレベル

[x] Lv1: ぱっとみて違和感がないかチェックして Approve する
[ ] Lv2: 仕様レベルまで理解して、仕様通りに動くかある程度検証して Approve する
[ ] Lv3: 実際に環境で動作確認したうえで Approve する

レビュー優先度

[ ] すぐに見てもらいたい ( hotfix など ) 🚀
[ ] 今日中に見てもらいたい 🚗
[ ] 今日〜明日中で見てもらいたい 🚶
[x] 数日以内で見てもらいたい 🐢

画像 / 動画

https://github.com/user-attachments/assets/d787d906-1c27-4803-afe3-fffdd3071e14

確認したこと

[x] デバッグ機能から強制アップデートを有効にしてスナックバーが表示されること
[x] テストが通ること

動作確認手順

動画の手順通り

エミュレータ、シミュレータでアプリを起動して Shift + D でデバッグ画面へ移動
Enable force update をタップ

備考

github-actions[bot] commented 3 months ago

Visit the preview URL for this PR (updated for commit 0ac2cd6):

https://flutter-mobile-project-template-catalog--pr310-improve-ovzac7dx.web.app

_{(expires Wed, 21 Aug 2024 04:56:35 GMT)}

_{🔥 via Firebase Hosting GitHub Action 🌎}

_{Sign: 9ea56735a63d07a7cfe62eb204b0528284c37c23}

github-actions[bot] commented 3 months ago

Ready for review :rocket:

tatsutakein commented 3 months ago

丁寧にご確認いただきありがとうございました。マージいたします 🙋