直近の CoderDojo 開催情報を表示する

[chicaco]

背景

直近の CoderDojo 開催情報を表示したい

やりたいこと

• 直近のイベント情報を集計するtaskの追加(仕切り直し) #459 で収集した情報を元に、直近の CoderDojo 開催情報を CoderDojo サイトのイベントページに表示する

cf #258

このPRでやること

• CoderDojo トップページ イベントページを新設して、直近の CoderDojo 開催情報を追加する
• HomeController EventsController の show メソッドで、直近の CoderDojo イベント情報リストを設定する
• view を定義する (app/views/home/show.html.haml への追記 app/views/events/show.html.haml, app/views/shared/_upcoming_events.html.haml の追加、CSS 追記)
• 近日開催イベント(upcoming_events)T に event_title を追加する
• rake タスクの rspec を修正する

やらなかったこと

特になし

レビューポイント

• 直近の CoderDojo 開催情報の表示方法
• 直近の CoderDojo 開催情報の表示内容

困ってること

特になし

[chicaco]

ひとまず、こんな感じで表示するようにしてみました。
「必要十分な情報を掲載すること」を目標にしているので、シンプルです。

[chicaco]

「CoderDojo.jp への掲載依頼をいただいている Dojo の内、connpass と doorkeeper で公開されているイベント情報のみ収集して表示しています。」という注記があった方がいいかな、という気がします。
全イベントを網羅しているという誤解を受けないために。
いかがでしょう？ > @yasulab

[yasulab]

@chicaco トップページに追加する場合は、デザイン上の全ての課題を洗い出してクリアする必要があるので、PR をマージするのにかなり時間がかかってしまいそうです >< 💦 (トップページを大きく変更する PR は基本的にマージされないと思っていただけると嬉しいです)

Issue にも記述した通り、ひとまずは /events などの別ページで表示してもらえると助かります (＞人＜;)💦
cf. #258

やりたいこと

/events ページで今後開催されるイベント情報を表示したい。当該ページでは次の情報を参照できる:

• これから開催される Dojo 情報のみを表示
• 遠い Dojo 情報を知っても見学できないので、できれば都道府県ごとに表示できると良い
• ユーザーは、近隣の Dojo 情報を参照し、都合の良い日に見学申請を申し出る
• cf. 統計情報ページ: https://coderdojo.jp/stats

[chicaco]

トップページに追加する場合は、デザイン上の全ての課題を洗い出してクリアする必要があるので、PR をマージするのにかなり時間がかかってしまいそうです >< 💦 (トップページを大きく変更する PR は基本的にマージされないと思っていただけると嬉しいです)

失礼しました。
「表示すること」の優先具合を間違えましたね。

トップページは元に戻して、/events ページに切り出します。 🙇‍♀

[chicaco]

独立したイベントページを追加しました。

遠い Dojo 情報を知っても見学できないので、できれば都道府県ごとに表示できると良い

とのことでしたが、スマートフォン版の Dojo 情報も地方ごとになっていますので、それに合わせる方向で地方ごとの表示にしてみました。

[yasulab]

とのことでしたが、スマートフォン版の Dojo 情報も地方ごとになっていますので、それに合わせる方向で地方ごとの表示にしてみました。

あー、これはスマホ版を都道府県ごとに寄せる方が良いですね 👀 💭 地方区分にしていた理由は、単に当時のCoderDojo数だとその粒度で十分だったという程度で、今のCoderDojoコミュニティの規模では都道府県別の粒度の方が良いかなと考えています ;)

[chicaco]

あー、これはスマホ版を都道府県ごとに寄せる方が良いですね 👀 💭 地方区分にしていた理由は、単に当時のCoderDojo数だとその粒度で十分だったという程度で、今のCoderDojoコミュニティの規模では都道府県別の粒度の方が良いかなと考えています ;)

なるほど！ 💡
後ほどイベントページを都道府県別に修正します。

スマートフォン版の Dojo 情報を都道府県別にするのは、別 issue & PR でよいでしょうか？

[yasulab]

スマートフォン版の Dojo 情報を都道府県別にするのは、別 issue & PR でよいでしょうか？

はい！その方向でお願いできると嬉しいです！！(＞人＜ )✨

[nalabjp]

app/views/shared/upcoming_events.html.hamlはeventsでしか使われていないようなので、app/views/events/配下に置いてもらえると良いかと思います。
それ以外はLGTMです

[chicaco]

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

app/views/shared/upcoming_events.html.haml は events でしか使われていないようなので、app/views/events/ 配下に置いてもらえると良いかと思います。

ファイルの配置を変更しました。
イベント情報の表示を 地方別→都道府県別 に変更しましたので、再レビューお願いします。

[chicaco]

都道府県別表示に変更しました。 > @yasulab
開催予定のないところは都道府県名も表示しません。

[yasulab]

都道府県別表示に変更しました。

@chicaco ありがとうございます! 😻✨

開催予定のないところは都道府県名も表示しません。

ここはちょっとどう表示した方が良いか僕自身も決めきれていませんが、ひとまずはこの方向で行きましょう...!! d(￣ ￣)✨ もし気になる部分があれば、別の PR で改善すれば良いかなと考えています 😉 ✨

[yasulab]

Heroku Review Apps でチェックしてみようと思ったけど、そもそもデータが無かった... 🙈 💦

cf. https://coderdojo-japan-pr-460.herokuapp.com/events

[chicaco]

Heroku Review Apps でチェックしてみようと思ったけど、そもそもデータが無かった... 🙈 💦

本番環境では収集済みなのですけれどね。 😅

開催予定のないところは都道府県名も表示しません。

ここはちょっとどう表示した方が良いか僕自身も決めきれていませんが、ひとまずはこの方向で行きましょう...!! d(￣ ￣)✨ もし気になる部分があれば、別の PR で改善すれば良いかなと考えています 😉 ✨

一旦並べてみたのですが、東北や四国で「開催予定なし」の県が続いていて、「なし」のインパクトが強いかなと感じたので県名の表示もしないようにしました。
表示の仕方が洗練されてから改善してもよいかな～、と思っています！

[AnaTofuZ]

HerokuReviewApps特性上DB関連のrake taskはapp.jsonに書かないといけないので、そのあたりで引っかかってるかもしれません!

coderdojo.jp/app.json

Line 81 in 61ecd77

"postdeploy": "bundle exec rails db:schema:load && bundle exec rails db:seed && bundle exec rails dojos:update_db_by_yaml && bundle exec rails dojo_event_services:upsert && bundle exec rails assets:precompile RAILS_ENV=staging"

[nalabjp]

👍

[yasulab]

良さそうです！！準備の良い段階でマージしていただければ (＞人＜ )✨