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

連名で登録している道場もカウント可能にする #860

Merged
merged 6 commits into from Jun 21, 2020

Conversation

yasulab
Copy link
Member

@yasulab yasulab commented Jun 21, 2020

Fix #610

連名で登録している道場もカウントできるようにしました! d( ̄  ̄)✨
@chicaco デザインの部分以外はできたので (デザイン部分もデザイン案は既に上がっていてあとは実装するだけなので)、一旦レビューお願いできると嬉しいです...!! (>人< )✨

image

image

やること

やらないこと

パッと見てどう対応するとベターなのか思いつかったので、Issue として切り出して一旦後回しにしました >< 💦 (Issue: #862 )

  • 『集計対象の道場数』に連名道場をうまい感じに反映する (集計対象の場合のみ)
    • 集計できている連名道場 (西宮・梅田大阪狭山・本町) と、そうでない連名道場 (しまね) があるため、その点を考慮して下記『集計対象の道場数』をカウントする必要がありそうです 👀 💭 image

当該コード部には TODO コメントを追記 📝

    # 集計方法と集計対象
    # TODO: 'DISTINCT dojo_id' cannot track joint-registrated dojos
    #   cf. https://github.com/coderdojo-japan/coderdojo.jp/issues/610
    @aggregated_dojos   = DojoEventService.count('DISTINCT dojo_id')

@yasulab yasulab added 急ぎじゃないよ Make something better but not rushed. 統計情報 Tracking event record function via APIs: https://coderdojo.jp/stats labels Jun 21, 2020
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.

counter 方式、シンプルでよさそうですね。
(自分は上位クラスを作るアプローチを考えていましたが、ぱっとは整理できなかったので)

WIP なので一旦コメントに留めておきますが、問題ないと思います!

@yasulab
Copy link
Member Author

yasulab commented Jun 21, 2020

早速のレビューありがとうございます!(>人< )💖
後はデザイン部分だけなので、デザイン案を参考に仕上げてみますね 🛠💨✨

@yasulab yasulab changed the title WIP: 連名で登録している道場もカウント可能にする 連名で登録している道場もカウント可能にする Jun 21, 2020
@yasulab
Copy link
Member Author

yasulab commented Jun 21, 2020

デザイン案に沿って実装してみました!d( ̄  ̄)✨
image

Tooltip があるとより Descriptive になりそうなので、簡単な Tooltip も導入しておきました 😉
image

テストが通ったらこちらマージしますね! 🛠💨✨

@yasulab
Copy link
Member Author

yasulab commented Jun 21, 2020

こちら CI も通ったのでマージしますね! 🚀🆕✨
デザイン案およびレビュー対応ありがとうございました...!! (>人< )💖

@yasulab yasulab merged commit 1fcde34 into master Jun 21, 2020
@yasulab yasulab added this to ✅Done in coderdojo.jp 開発ボード via automation Jun 21, 2020
@yasulab yasulab deleted the add-counter-column-to-dojo-for-joint-registration branch June 21, 2020 14:05
yasulab added a commit that referenced this pull request Jun 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
急ぎじゃないよ Make something better but not rushed. 統計情報 Tracking event record function via APIs: https://coderdojo.jp/stats
Projects
Development

Successfully merging this pull request may close these issues.

📊 連名で登録されている Dojo も統計情報で集計できるようにしたい
3 participants