[GH-439] domain_model に移行#540
Merged
Merged
Conversation
Compare pubspec.yaml and pubspec.lock. Packages with different version notations
|
1 similar comment
Compare pubspec.yaml and pubspec.lock. Packages with different version notations
|
…モード設定を実装 - 新しいビルド設定を `build.yaml` に追加 - `pubspec.yaml` に依存関係を追加 - `force_update_settings_state.dart` と `force_update_target_version.dart` を新規作成 - `maintenance_mode_settings_state.dart` を新規作成し、メンテナンスモードの状態を管理 - `version_string.dart` を追加し、バージョン管理のための拡張を実装
- `build.yaml` での空白を整理 - `force_update_settings_state.freezed.dart`、`force_update_target_version.freezed.dart`、`maintenance_mode_settings_state.freezed.dart` でのエラーメッセージの末尾にカンマを追加 - 各クラスの `copyWith` メソッドのフォーマットを統一
yumemi-team-review-requester
Bot
requested review from
a team and
r0227n
and removed request for
a team
|
Ready for review 🚀 |
blendthink
suggested changes
Jun 4, 2025
Contributor
blendthink
left a comment
blendthink
left a comment
There was a problem hiding this comment.
@karas1910
おそらくパッケージバージョンを揃える必要があるのと、その後の警告対応などが必要かなと思いますので、ご確認お願いします🙏
- `pubspec.yaml` で `freezed_annotation` を ^2.4.4 から ^3.0.0 に変更 - `freezed` を ^2.5.7 から ^3.0.6 に更新 - `test` のバージョンを ^1.25.8 から ^1.25.15 に変更
r0227n
reviewed
Jun 4, 2025
| part 'force_update_settings_state.freezed.dart'; | ||
|
|
||
| @freezed | ||
| class ForceUpdateSettingsState with _$ForceUpdateSettingsState { |
Contributor
There was a problem hiding this comment.
@karas1910
freezedは3系から sealed or abstract が必要になっているので、下記のドキュメントの確認お願いします。
https://github.com/rrousselGit/freezed/blob/master/packages/freezed/migration_guide.md
Contributor
Author
…ceModeSettingsStateを抽象クラスに変更 - `ForceUpdateSettingsState`、`ForceUpdateTargetVersion`、`MaintenanceModeSettingsState`を抽象クラスとして定義
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
概要
close: #439
アプリの強制アップデートとメンテナンスモード機能に必要なドメインモデルを追加しました。
主な追加項目：
ForceUpdateSettingsState- 強制アップデート設定の状態管理ForceUpdateTargetVersion- 強制アップデート対象バージョンの管理MaintenanceModeSettingsState- メンテナンスモード設定の状態管理VersionString- バージョン文字列の比較機能付きextension typeレビュー観点
ForceUpdateSettingsState.isForceUpdateEnabled）の妥当性VersionStringのバージョン比較演算子の実装レビューレベル
レビュー優先度
画像 / 動画
見た目に関する変更がないため省略します。
確認したこと
VersionStringのバージョン比較が正しく動作すること動作確認手順
備考
このPRではドメインモデルの追加のみで、実際の機能実装は別PRで行います。