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

奈良・生駒・田原本をイベント収集対象に追加 #481

Merged
merged 1 commit into from
Jun 11, 2019

Conversation

chicaco
Copy link
Contributor

@chicaco chicaco commented Jun 10, 2019

背景

奈良・生駒・田原本は、共通のイベント情報収集サイトを利用している。
https://coderdojo-nara-ikoma.connpass.com/

cf. #473

やりたいこと

奈良・生駒・田原本をイベント収集対象に追加して、以下を実現する。

  • 統計情報で開催回数、参加人数を収集できる
  • 近日開催の道場情報に掲載する

このPRでやること

  • db/dojo_event_services.yaml に、代表とする奈良の dojo に対して奈良・生駒・田原本の connpass サイトを追記する
  • 近日開催の道場情報の Dojo 名には、「奈良・生駒・田原本」と表示する

やらなかったこと

  • 複数 Dojo で共通の dojo_event_service を利用している場合の永続的&汎用性のある対応 (まだよい方法を見付けていない)

レビューポイント

  • 対象の dojo_event_service の特定には、プロバイダ種別の 'connpass' とグループID '2617' を使用し、dojo_id や dojo_event_service_id には影響を受けないようにしました

困っていること

特になし

@chicaco chicaco self-assigned this Jun 10, 2019
@chicaco chicaco changed the title 奈良・生駒・田原本をイベント収集対象に追加 [WIP] 奈良・生駒・田原本をイベント収集対象に追加 Jun 10, 2019
@chicaco
Copy link
Contributor Author

chicaco commented Jun 10, 2019

近日開催の道場情報の Dojo 名に「奈良・生駒・田原本」と表示する方法でよいかの了承待ちです。

@chicaco chicaco changed the title [WIP] 奈良・生駒・田原本をイベント収集対象に追加 奈良・生駒・田原本をイベント収集対象に追加 Jun 11, 2019
@chicaco
Copy link
Contributor Author

chicaco commented Jun 11, 2019

了解いただいたので、WIP を外します。

@chicaco chicaco requested a review from nalabjp June 11, 2019 00:08
@chicaco chicaco marked this pull request as ready for review June 11, 2019 00:08
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.

LGTM
個別対応なので今回はこのままで良いと思いますが、
似たようなケースが増えてきたらdojo_event_servicesかその関連にalias_nameみたいなカラムを用意したい感じですね。

@chicaco
Copy link
Contributor Author

chicaco commented Jun 11, 2019

個別対応なので今回はこのままで良いと思いますが、
似たようなケースが増えてきたらdojo_event_servicesかその関連にalias_nameみたいなカラムを用意したい感じですね。

はい、よほど特殊なケースでなければ個別対応は辞めたい派です。
「やらなかったこと」にも記載した通り、統計情報&近日開催の道場情報収集の他の課題も整理して、永続的&汎用性のある対処を検討したいと思っております。

@chicaco chicaco merged commit e36f028 into master Jun 11, 2019
@chicaco chicaco deleted the add_nara-ikoma_event_service branch June 11, 2019 03:11
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