daichikuwa0618 / flutter-weather-app

Dummy Weather App Built with Flutter

1 stars 0 forks source link

[Session/7] freezed による JSON シリアライズの導入 #20

Closed daichikuwa0618 closed 5 months ago

daichikuwa0618 commented 6 months ago

課題

close #8

対応箇所

[x] json_serializable を利用して、独自のクラスから API のリクエストを作成する
[x] json_serializable を利用して、API のレスポンスを独自のクラスに変換する

※ freezed を利用しても問題ありません

動作

動作に変更なし

github-actions[bot] commented 6 months ago

Ready for review :rocket:

daichikuwa0618 commented 6 months ago

19 の b747df0bb673ff9606bdb14dbb28cff0a554a46f のコミットで、 `GetWeather` に定義していた `_Response` を廃止して直接 `factory Weather.fronJson` を定義するよう移動した関係で、本 PR も影響を受けています。

下のコミットを活かしたまま rebase は厳しかったので、_Response に関わるコミットは捨てて改めて 2b8050a54174cbdfefe1b2b3d6de7a08a1dcbd08 にて Weather に freezed を使うように実装し直した形です。

_WeatherConditionConverter はそのままですので、基本的な実装は変わっていません。