yumemi-inc / flutter-mobile-project-template

MIT License

37 stars 7 forks source link

[Improve]: メンテナンス関連の実装を features/maintain パッケージに切り出した #317

Closed macneko-ayu closed 3 months ago

macneko-ayu commented 3 months ago

概要

304

メンテナンス関連の実装を features/maintain パッケージに切り出した
ついでに l10n 対応をした

レビュー観点

メンテナンスモードの表示ができるか
l10n 対応の訳が妥当か

レビューレベル

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

レビュー優先度

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

画像 / 動画

言語	Before	After
日本語
英語

確認したこと

[x] デバッグ画面からメンテナンスモード画面に遷移できることを確認した
[x] シミュレータの言語設定を英語/日本語と切り替えて、言語設定に沿ったローカライズがされていることを確認した
備考

github-actions[bot] commented 3 months ago

Ready for review :rocket:

github-actions[bot] commented 3 months ago

Visit the preview URL for this PR (updated for commit 42eb668):

https://flutter-mobile-project-template-catalog--pr317-improve-g8reie0k.web.app

_{(expires Tue, 20 Aug 2024 08:32:56 GMT)}

_{🔥 via Firebase Hosting GitHub Action 🌎}

_{Sign: 9ea56735a63d07a7cfe62eb204b0528284c37c23}

macneko-ayu commented 3 months ago

@tatsutakein レビューありがとうございます！このPRは移植だけをスコープとすることにして、指摘いただいた点は別PRで対応します！