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.

Sign up for GitHub

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

Jump to bottom

add_2019-10_Facebook_event #632

Merged
merged 1 commit into from
Nov 17, 2019
Merged

add_2019-10_Facebook_event #632

merged 1 commit into from
Nov 17, 2019

Conversation

Yuppymam
Copy link
Contributor

やった事

  • 2019年10月分の Facebook イベントを追加しました🎉
  • name: facebook で検索して作業しているので、私が見やすいようにコメントの位置をいくつか変更しました。
  • CoderDojo北九州 は第三回から connpass で運用されているようなので、第二回までの Facebook event は残して、それ以降は connpass に移行します。

気になった事

台風で中止になった Dojo が３つありました。イベントページはあるのでコメントアウトにしています。 API 復活したら集計対象になってしまうのですが、ここの対応はどうしましょう...

@yasulab yasulab temporarily deployed to coderdojo-japan-pr-632 November 15, 2019 12:42 Inactive
@yasulab yasulab temporarily deployed to coderdojo-japan-pr-632 November 15, 2019 12:43 Inactive
@Yuppymam
Copy link
Contributor Author

@chicaco お手隙の際にご確認よろしくお願いします📄✨

@Yuppymam Yuppymam force-pushed the 2019_10_facebook_event branch from d6c3daf to bad3b4c Compare November 15, 2019 13:54
@Yuppymam Yuppymam requested a review from chicaco November 15, 2019 14:32
@Yuppymam Yuppymam self-assigned this Nov 15, 2019
@chicaco
Copy link
Contributor

chicaco commented Nov 17, 2019

PR 作成ありがとうございます！

気になった事

台風で中止になった Dojo が３つありました。イベントページはあるのでコメントアウトにしています。 API 復活したら集計対象になってしまうのですが、ここの対応はどうしましょう...

一旦、コメントアウトのままで進めましょう。

@chicaco
Copy link
Contributor

chicaco commented Nov 17, 2019 •
edited
Loading

📝 デプロイ後にやることメモ

facebook イベント ＋ イベント収集先変更分 ＋ 10 月より掲載追加した dojo のイベント収集

  • 10 月分の facebook イベントの収集
  • 北九州(dojo_id: 144)の再収集、2018年11月以降
  • 木更津(dojo_id: 83)の再収集、2019年10月以降
  • 昭島(dojo_id: 228)の収集、2019年7月以降
  • ナカマ(dojo_id: 229)の収集、2019年9月以降

以下は対応不要

  • 浜田は 2019年11月イベントより切り換えのため、収集なし
  • 戸田公園(dojo_id: 227) の収集、2019年8月以降 → 済み
  • みきは 2019年12月よりスタート予定のため収集なし
  • 板橋@桜川はイベント管理が独自のため、収集なし

chicaco
chicaco approved these changes Nov 17, 2019
View reviewed changes
Copy link
Contributor

@chicaco chicaco left a comment

Choose a reason for hiding this comment

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

こちら、問題なさそうです！
マージしますね。

@chicaco chicaco merged commit 3837351 into master Nov 17, 2019
@chicaco chicaco deleted the 2019_10_facebook_event branch November 17, 2019 14:10
@chicaco
Copy link
Contributor

chicaco commented Nov 17, 2019

  • 昭島(dojo_id: 228)の収集、2019年7月以降

イベント収集先がたまみらと共用のため、収集エラー → 別 PR で修正します！

@chicaco chicaco mentioned this pull request Nov 17, 2019
Merged
2 tasks
@yasulab
Copy link
Member

yasulab commented Nov 17, 2019

対応ありがとうございます...!! (＞人＜ ) 💖✨

@Yuppymam Yuppymam mentioned this pull request Dec 16, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chicaco chicaco chicaco approved these changes

Assignees

@Yuppymam Yuppymam

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Yuppymam @chicaco @yasulab