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.
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
イベント履歴収集スクリプトの改修 #375
イベント履歴収集スクリプトの改修 #375
Conversation
日/月ごとの取得でなく、dojo ごとにできるだけ一括取得に (connpass は期間指定の場合、月or日 の組み合わせにする必要がある)
PR ありがとうございます! (>人< )✨
cf. #364 |
やらなかったことは Check Box 形式じゃない方が良さそうですね 👀 「この PR ではやらない」と決めたことなので。 一方で、1つの PR にテストも含めていて欲しいので、
までやってもらえると嬉しいです (>人<;) 💦 |
レビューポイントあるのはいいですね! どこを見ればいいかの参考になって助かります 😆✨ |
ひとまず PR コメントを訂正してみました。 ✨ @yasulab facebook のグループイベントについて ご意見、お願いします。 |
なるほど 🤔 @chicaco さんとしてはどのように進めていくと良いと考えていますか? この PR にその機能も含めてしまうべきですかね?それとも別 Issue / 別 PR で対応した方が良いですかね? 💭 |
なるほど!その方向が良さそうですね!😆✨ まとめる必要性もないですし、少しずつリリースできた方がバグが出たときも対応しやすいので、別 PR / 別 Issue で進めていきましょう d( ̄  ̄)✨ |
上記issueコメントについての情報がないのでまずはこちらを整理したほうがいいと思いました。 faraday_middlewareの必要性については |
Description の「完了条件」と「やったこと」を別々に分ける必要がないので、まとめておきました ;) cc/ @chicaco |
コメントありがとうございます。 > @nalabjp @yasulab
失礼いたしました。
connpass, doorkeeper (もちろん static_yaml も) については、ローカル環境で動確取れており、
dooekeeper もレート制限あるようです。
お手数お掛けしました。 🙇♀️ ひとまず、rspec 追加します。 |
RSpec 追加しましたので、[WIP] は削除しました。 |
説明を見直しました。 |
d( ̄  ̄)✨ |
Facebook のグループイベント取得について、作戦変更することになりました。 ということで、Facebook イベントの情報収集は、static_yaml 形式で暫定運用します。 イベント履歴収集スクリプトも、この方針に沿って再修正します。 → 再度 WIP に戻し、「このPRでやること」を訂正します。 |
本番環境DBのイベント履歴レコードを確認し、収集できていない Facebook イベント分を static_yaml 化する必要がある。 |
MEMO 理由: |
Dumper.io から取得した DB バックアップ 20190211_230105.sql.gz に、static_yaml 分のイベント履歴が含まれていない。 過去の実績が変わってしまいますが、「より正しく」なることはヨシとしてよいでしょうか? > @yasulab |
良いと思います! d( ̄  ̄)✨ 数字が大きく下方修正するようであれば一般に周知する必要性も出てくると思うので、その可能性も加味しておくと良さそうですね ;) |
了解です。 facebook でエラーが出始める前までは登録できていた static_yaml 分が今は登録できていない状態なので、今現在表示されている履歴は 2018/11 時点の数字より「少なく」なっているかと。 |
dojo_event_services テーブルに、facebook 系のみ、おそらく削除したかったレコードが残っていそう。
'/?ref=page_internal' 付きの url を持つレコードが残っていても、イベント履歴収集では今の処悪さはしていなさそうではあるが...。 |
Facebook イベント履歴の収集について、グラフAPI経由を止め、facebook_event_histories.yaml からの読み込みに変更しました。 |
お疲れ様です!! 😆🎉✨ 後で見ますね〜 👀 (と言いつつ、遅いと今週末ぐらいになりそうです... >< 💦 ) |
以下、備忘のためのメモです。 (1) 2018 年分の実績収集マージ&リリース後、2018/04~ の期間を指定してイベント履歴収集スクリプトを実行していただければ、2018 年の実績が揃います。「connpass 側の検索処理が修正されていれば」という条件付きですが。 (2) スクリプト実行日/間隔スクリプト実行前に yaml にイベント情報を追記する必要があるので、現状通り週次の月曜朝実行ですと、週末開催の facebook 分を収集対象にするのは厳しいかもしれません。 しばらく手作業による追記を行うとして、月曜に yaml 作成 → 火曜朝に収集、が現実的な気がします。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
良さそう! lib/tasks/docs/statistics_aggregation.md
のドキュメントは README か /docs に置いておいたほうが見つけやすそうかなと思いました ✅ 👀 💭
マージのタイミングはお任せしますね! chicaco さんの都合の良いタイミングにマージしていただければ d( ̄  ̄)✨
(間違って request changes を押してしまった...orz)
Fix #258 (直近の CoderDojo 開催情報を表示したい)
背景
統計スクリプトが本番環境でも失敗しているので、直さないといけない
※ 2018/12/03 は成功していたが、その後失敗
やりたいこと
(2018/12/02 までは収集済み)
現 : 週 or 月の期間 × dojo ごと収集 → 新 : dojo × 一括 or 指定可能なまとまった期間 ごと収集
Fix #371
このPRでやること
アプリレビューで承認されたアカウントのトークンで、facebook のイベント履歴収集の動作確認facebook イベントのグラフAPI経由での履歴収集はスコープ外にした Facebook Events の情報収集を自動化したい #393やらなかったこと
特になし
レビューポイント
困ってること
cf. https://developers.facebook.com/docs/groups-api#app-review
🆕 追記: 本機能リリース後の反響 😻✨