refactor(force_update): 強制アップデート関連処理のリファクタリング - Githubissues

yumemi-inc / flutter-mobile-project-template

MIT License

37 stars 7 forks source link

refactor(force_update): 強制アップデート関連処理のリファクタリング #323

Closed tatsutakein closed 3 months ago

tatsutakein commented 3 months ago

概要

305

※ 引き続き対応を進めるため、敢えて close をつけていません。

強制アップデート関連処理のリファクタリングを実施します。この PR では、以下の作業を実施しています。

文字列で取り扱っていたバージョン情報を VersionString に変更
バージョンリストの取得処理を VersionString に移動
対象のプラットフォームからバージョン情報を取得する処理を ForceUpdateTargetVersion 内に移動
バージョン比較の実装を VersionString に移動

レビュー観点

変更に過不足がないか
デグレ等がなさそうか
一旦今までの挙動と同じままになるよう、必要最低限の修正に留めています 🙏
より良い実装方法などあればご教授いただけますと幸いです

レビューレベル

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

レビュー優先度

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

画像 / 動画

Before	After	Design

確認したこと

[x] アプリが起動すること
[x] デバッグ機能から引き続き強制アップデートの有効化が実施できること

動作確認手順

エミュレータ、もしくはシミュレータにてアプリを起動
Shift + D でデバッグ機能から強制アップデートを有効にし、スナックバーが表示されること

備考

github-actions[bot] commented 3 months ago

Visit the preview URL for this PR (updated for commit 55bd242):

https://flutter-mobile-project-template-catalog--pr323-improve-oxpj6vh3.web.app

_{(expires Wed, 28 Aug 2024 06:01:24 GMT)}

_{🔥 via Firebase Hosting GitHub Action 🌎}

_{Sign: 9ea56735a63d07a7cfe62eb204b0528284c37c23}

github-actions[bot] commented 3 months ago

Ready for review :rocket:

tatsutakein commented 3 months ago

ご確認いただきありがとうございました！マージいたします 🙆