Skip to content

イベント履歴収集スクリプトでプロバイダ指定を可能に #392

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

Merged
merged 1 commit into from
Mar 6, 2019

Conversation

chicaco
Copy link
Contributor

@chicaco chicaco commented Mar 4, 2019

背景

毎週月曜に前週分のイベント履歴を収集しているが、facebook のみ 2018/04 以降収集できていなかった。
このような場合、対象期間とプロバイダを指定してイベント履歴を収集し直したい。

やりたいこと

  • イベント履歴収集スクリプトで期間だけでなくプロバイダを指定して実行可能にする
  • rake タスクの usage を修正する + ファイル配置を見直す

Fix #378

このPRでやること

  • タスクの引数にプロバイダを追加し、対象プロバイダのみイベント履歴を収集できるようにする
  • usage を修正する
  • 対象プロバイダ判定の rspec 追加する

やらなかったこと

特になし

レビューポイント

  • 対象プロバイダの判定

困ってること

特になし

継続検討すること

@chicaco chicaco self-assigned this Mar 4, 2019
@chicaco
Copy link
Contributor Author

chicaco commented Mar 5, 2019

こちらも実装 & RSpec 追加、完了しました。
お手隙のときにレビューお願いします。 🙏 > @yasulab @nalabjp

@yasulab yasulab requested a review from nalabjp March 5, 2019 01:23
Copy link
Member

@nalabjp nalabjp left a comment

Choose a reason for hiding this comment

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

良さそうです!👍

これとは別で良いと思うんですが、facebookが実質使えなくなってしまったので、インタフェースを閉じた方が良さそうかなと思いました。

@yasulab
Copy link
Member

yasulab commented Mar 5, 2019

facebookが実質使えなくなってしまった

これちょっとアイデア待ちですね 🤔💭

個人的には Facebook アプリに登録してもらうのはそれなりにコストがかかるので何か良い方法がないかなぁと考えていますが (できれば Doorkeeper や connpass のような一般公開情報だけに絞って取得したいと考えているのですが)、もし Facebook アプリでやるとしたら、 https://coderdojo.jp/kata#support【協力依頼】 というセクションを用意して、そこで Facebook アプリを承認してもらうという感じですかね。

今の所このぐらいしかアイデアがない状況です 😓.oO(もっといい方法ないかなぁ)

@chicaco
Copy link
Contributor Author

chicaco commented Mar 5, 2019

facebookが実質使えなくなってしまった

これちょっとアイデア待ちですね 🤔💭

はい。
まずは 2018 年の履歴を収集することを優先して、yaml 投入方式で回避している、という認識です。
そのため、インタフェースは残し、期間指定もできるようにしました。

今の所このぐらいしかアイデアがない状況です 😓.oO(もっといい方法ないかなぁ)

Facebook のイベント履歴、よい策があれば自動収集したいので Issue 立てておきます。

@chicaco chicaco merged commit dd3bd01 into master Mar 6, 2019
@chicaco chicaco deleted the aggregation_specify_provider branch March 28, 2019 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants