Skip to content
This repository has been archived by the owner on Dec 6, 2023. It is now read-only.

詳細な予報結果の表示 #20

Merged
merged 6 commits into from
Nov 27, 2023
Merged

詳細な予報結果の表示 #20

merged 6 commits into from
Nov 27, 2023

Conversation

warahiko
Copy link
Owner

課題

close #7

対応箇所

  • 使用する API を fetchThrowsWeather() から fetchWeather() に変更する
  • API から受け取った天気状況・最低気温・最高気温を画面に表示する

動作

expected actual

@warahiko warahiko marked this pull request as ready for review November 22, 2023 08:51
@github-actions GitHub Actions
Copy link

Ready for review 🚀

blendthink
blendthink approved these changes Nov 27, 2023
View reviewed changes
Copy link
Collaborator

@blendthink blendthink left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM です！
LGTM

lib/forecast_view.dart Show resolved Hide resolved
lib/main_screen.dart Show resolved Hide resolved
lib/forecast.dart Show resolved Hide resolved
@warahiko warahiko merged commit 68ea87d into main Nov 27, 2023
3 checks passed
@warahiko warahiko deleted the feature/fetch_with_json branch November 27, 2023 03:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@blendthink blendthink blendthink approved these changes

Assignees

@warahiko warahiko

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Session6
2 participants
@warahiko @blendthink