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.

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

たまみらと昭島はイベント収集サイト共用 #635

Merged
merged 1 commit into from
Nov 18, 2019

Conversation

chicaco
Copy link
Contributor

@chicaco chicaco commented Nov 17, 2019

背景

たまみら(dojo_id: 221)と昭島(dojo_id: 228)がイベント管理サイトを共用しているため、イベント履歴収集時に重複エラーが発生した。
https://coderdojo-tamamira.connpass.com/
⇒ 奈良・生駒・田原本と同様の処理にする

(cf. #632)

このPRでやること

  • db/dojo_event_services.yaml で、代表とするたまみらの情報を有効とし、昭島 の情報を削除
  • 近日開催の道場情報の Dojo 名には、「昭島・たまみら」と表示する

やらなかったこと

特になし

困っていること

特になし

@Yuppymam
Copy link
Contributor

@chicaco 修正プルリクありがとうございます✨💖

if 文のところ、今後もこのような処理が増える可能性がありそうなら case 文の方が見やすいのかな... ?とも思いましたが、いかがでしょうか?💦

あと、集計の仕方を私もマスターしたいです😭chicaco さんが集計に使ったコマンドが知りたいです😆

@chicaco
Copy link
Contributor Author

chicaco commented Nov 18, 2019

レビューありがとうございます。

if 文のところ、今後もこのような処理が増える可能性がありそうなら case 文の方が見やすいのかな... ?とも思いましたが、いかがでしょうか?💦

情報共有不足で済みません。

実は抜本的な修正が必要だと認識していて、対応を検討中です。連名で登録されている Dojo への対応の中で合わせて解決したいな、と目論んでいます。ですので、if の分岐はこれ以上増やさない見込みなので、今回はこちらでよいかなと考えました。

cf. #481 奈良・生駒・田原本をイベント収集対象に追加
 #610 連名で登録されている Dojo も統計情報で集計できるようにしたい

あと、集計の仕方を私もマスターしたいです😭chicaco さんが集計に使ったコマンドが知りたいです😆

統計情報収集用コマンドは rake statistics:aggregation コマンドを使用しています。
使い方は こちら に記載しています。

例えば、facebook の 2019 年 10 月分を収集したいときは、期間とプロバイダを指定して rake statistics:aggregation[201910,201910,facebook] と実行します。
追加した dojo のみ 2018 年 1 月分から収集したいときは、期間と dojo_id (仮に xxx とします) を指定して rake statistics:aggregation[201801,201910,,xxx] と実行します!

@Yuppymam
Copy link
Contributor

連名で登録されている Dojo への対応の中で合わせて解決したい

なるほど!了解しました😊👌✨
コマンドのドキュメントあったのですね!ありがとうございます😆

色々試して見ようと思います😉

こちらのプルリク問題なさそうです!🙆‍♀️

@chicaco chicaco merged commit 345e040 into master Nov 18, 2019
@chicaco chicaco deleted the tamamira_and_akishima branch November 18, 2019 15:24
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