ユーザーのプロフィール写真が半分ぐらいリンク切れしてる

[muratayusuke]

https://github.com/kumonos/kumonos.jp/pull/12/files みたいな修正で直りそうな気がする

[tomoima525]

これ、まじですね、対応します！

[tomoima525]

内部的にKoalaというライブラリ経由でAPI叩いてることもあり、 @muratayusuke にあげていただいた方針がとれない。
ちょっと調べたところ、runnerってactiveレコードも叩けるバッチ処理ができるgemがあるようなので、定期的にAPI叩いて画像を更新する処理を入れるようにします。

[muratayusuke]

@tomoima525

whoswhoplus/app/models/profile.rb

Line 65 in 85234f9

picture_url: me['picture'].try { |p| p['data'].try { |d| d['url'] } }

この辺でpicture_urlにセットするやつを、単純に"https://graph.facebook.com/#{me['id']}/picture"みたいに書いちゃえばいけそうな気がします！getパラメータで大きさ指定したりできますし

[tomoima525]

おお、説明あざす！なるほど、なるほど！これは確かにいけそう！

ちょっと思ったのは、これだと

whoswhoplus/app/controllers/home_controller.rb

Line 24 in 85234f9

profile = Profile.insert_or_update(api.get_object('/me','fields'=>'name,gender,picture.width(200).height(200)','locale' =>'ja_JP'), access_token)

の部分でFacebook Api一回叩いて、さらにユーザーの数だけApi叩くことになってちょっとボトルネックになりそうなんですね。
なるべくApi叩く回数を減らした方がいいかなーってことと、実は今の実装だとユーザーの友人が増えた時に更新に行かないんで、ログイン時にデータがある程度古かったら、Resque使ってバックグラウンドでユーザー画像を取得し直す方向に変更しやすた！

runner使うことも考えたんですが、わざわざバッチでやるほどの処理でもなさそうってことで、やめときました〜

[tomoima525]

STGに修正あげてみました。俺の画像クリックと、友人画像が最新になっているのが確認できると思います！ http://whoswhoplus-staging.kumonos.jp

[tomoima525]

@youcune コードチェックよろしくお願いします！

[youcune]

👍
Mergeしました！

[muratayusuke]

👍

[tomoima525]

ゾス！