Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

APIをMetaWeather APIからOpenWeather APIへ移行 #15

Merged
merged 25 commits into from
Oct 28, 2023
Merged

APIをMetaWeather APIからOpenWeather APIへ移行 #15

merged 25 commits into from
Oct 28, 2023

Conversation

h-yoshikawa44
Copy link
Owner

Issue 番号

closes #14

対応内容

  • 使用する API を MetaWeather API から OpenWeather API へ移行
    • API ルートを BFF 層にして、viewModel として返すようにした
    • ロケーション検索が微妙で、同国・同都市名（緯度経度違い）が複数返ることがあるので、国コードと緯度経度も併せて表示するようにした
    • 5日予報は、API の仕様上3時間単位の取得であるため、基本的には正午の天気をベースとした

@h-yoshikawa44
settings: OpenWeatherMap API向けの環境変数追加(#14)
88fd2d8
@h-yoshikawa44
feat: kyのベースインスタンスを継承して使うような形式にした(#14)
f9ed707
@h-yoshikawa44
feat: ロケーション検索APIのうちAPIルート->外部API用の処理を分離(#14)
fa32ce4
@h-yoshikawa44
feat: ロケーション検索機能のうち、FE->APIルート部分の処理をviewModelを使う方式に移行(#14)
4714286
@h-yoshikawa44
fix: コメントをより詳細にした(#14)
fda1301
@h-yoshikawa44
feat: APIルートで現在の天気情報取得APIを新たに作成(#14)
e5b0938
@h-yoshikawa44
feat: 現在の位置情報を管理するHooksを簡略化(#14)
a9fb9f4 
API移行に伴い、緯度経度から天気情報が取得できるようになったので、
単純に緯度経度を取得して保持するだけでよくなった。
@h-yoshikawa44
fix: クエリパラメータに関する型名と型ガード名を個別にした(#14)
5ed275f
@h-yoshikawa44
fix: APIレスポンスのコメントに単位のことも追記(#14)
23a917c
@h-yoshikawa44
feat: 現在の天気情報取得APIでviewModelを生成して返すようにした(#14)
b0766a4
@h-yoshikawa44
feat: メートル/秒をマイル/時に変換する関数作成(#14)
8b36d23
@h-yoshikawa44
feat: 角度から方角に変換する関数作成(#14)
6e3506a
@h-yoshikawa44
feat: キロメートルをマイルに変換する関数作成(#14)
a574919
@h-yoshikawa44
feat: 現在の天気情報取得処理を新しいAPI方式に移行した(#14)
5341831
@h-yoshikawa44
feat: 5日間の天気予報情報取得API作成(#14)
d8f8272
@h-yoshikawa44
feat: 5日の天気予報情報APIでviewModelを返すようにした(#14)
bf62169
@h-yoshikawa44
feat: 5日間の天気予報情報APIデータを使って表示するようにした(#14)
a343212 
最高気温、最低気温は後でロジックを直す。
@h-yoshikawa44
remove: API移行に伴い不要になったものを削除(#14)
608af9a
@h-yoshikawa44
feat: 5日予報で、最高気温・最低気温が抽出できるようにした(#14)
7a71eb2
@h-yoshikawa44
docs: README記載の使用APIをOpenWeather APIに移行(#14)
fba9975
@h-yoshikawa44
fix: ロケーション検索結果のkeyは緯度経度にするようにした(#14)
07ab699 
緯度経度だとかぶることがないので。
@h-yoshikawa44
feat: 同国・同都市名の結果が複数返ってきても判別できるよう緯度経度も表示するようにした(#14)
63b9a2a 
苦肉の策だよ...
@h-yoshikawa44
fix: process.browserは非推奨になったので、typeof windowに修正(#14)
9f093f9
@h-yoshikawa44
docs: API移行に伴い微妙に変わったのでスクショ更新(#14)
05d3e91
@h-yoshikawa44 h-yoshikawa44 added the enhancement New feature or request label Oct 28, 2023
@h-yoshikawa44 h-yoshikawa44 self-assigned this Oct 28, 2023
@vercel
Copy link

vercel bot commented Oct 28, 2023 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ch-weather-app ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 28, 2023 1:31pm

@h-yoshikawa44 h-yoshikawa44 merged commit 139fb72 into main Oct 28, 2023
2 checks passed
@h-yoshikawa44 h-yoshikawa44 deleted the feature/14-open_weather_map_api branch October 28, 2023 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@h-yoshikawa44 h-yoshikawa44

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[移行]MetaWeather APIからOpenWeatherMap APIへ移行したい
1 participant
@h-yoshikawa44