天気情報の取得方法を変更する

codeforchiba / feschibal

[Deprecated] 千葉市お祭りデータセンター

MIT License

1 stars 1 forks source link

Closed seitaro1227 closed 7 years ago

seitaro1227 commented 7 years ago

data/weather.jsonからお天気情報を取得するように修正しました。

動作確認用に2017-06-20に取得したお天気情報をdata/weather.jsonに置いています。

misopeso commented 7 years ago

これって、なんで data/weather.json から取得しているんでしたっけ？いまいち、何がやりたいのかよくわからないのですが。。。

ymits commented 7 years ago

@misopeso Lambdaでお天気API投げて取得したレスポンスをS3側のdata/weather.jsonに定期的に上書きするのだと思います。コミットされているdata/weather.jsonはテストまたは動作確認用だと思います。

自分はこんなイメージでした。

misopeso commented 7 years ago

@seitaro1227 ちなみに、API から取得したデータを公開されているリポジトリに含めるのは NG だと思いますよ。

misopeso commented 7 years ago

このブランチ、変な差分があるなーと思ったら切り方間違ってますね。 master ではなく develop から切ってください。

お手前でなければ、修正してほしいです。ブランチ名も TYPO しているし :)

seitaro1227 commented 7 years ago

@misopeso 承知しました。現在のdevelopにリベースし、ブランチ名を修正します。一旦fix/weather_sotreブランチは削除します。

誤 fix/weather_sotre 正 fix/weather_store