Skip to content

CoderDojo 呉を coderdojo.jp 掲載する #316

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

Closed
12 of 13 tasks
yasulab opened this issue Apr 5, 2018 · 11 comments
Closed
12 of 13 tasks

CoderDojo 呉を coderdojo.jp 掲載する #316

yasulab opened this issue Apr 5, 2018 · 11 comments
Assignees

Comments

@yasulab
Copy link
Member

yasulab commented Apr 5, 2018

CoderDojo 呉から coderdojo.jp への掲載依頼が来たので、これに対応したい。(申請内容は別途共有)

対応例1: cf2d579
対応例2: #259

cf. 作業メモの例 - OSS Gate

チェックリスト

  • 新規Dojoの追加方法を読んでみた?
  • Ruby/Rails の環境構築はできた?
    • Hint: トップページの表示には環境変数は不要 (なはず)
  • db/dojos.yaml に Dojo 情報を追記した?
    • Hint: be rails dojos:update_db_by_yaml
    • 掲載する情報は @yasulab まで問い合わせてもらえれば ;) 📨
  • DBからIDを取得して db/dojos.yaml に追記した?
    • Hint: be rails dojos:migrate_adding_id_to_yaml
  • ローカル環境で Dojo が表示できた?
  • 統計情報にイベント管理サービスを登録できた?
  • 本番環境に反映された?
  • 上記のドキュメントに加筆修正した方が良い箇所はある?

作業の完了条件

  • CoderDojo 呉が coderdojo.jp に掲載されている
  • 依頼されたロゴが表示されている
  • 掲載しているロゴは TinyPNG などで画像圧縮されている
  • リンク先が適切である
  • 集計対象にも登録されている (毎週月曜日に統計情報が更新される)
@aokabin
Copy link
Contributor

aokabin commented Apr 6, 2018

cf. 作業メモの例 - OSS Gate
を参考に見てみた、なるほど、GitHubのIssueにどんどんコメントとして残していく、それは良さそう!
ということでメモをガンガン残していくスタイルにする

上のチェックリストから進めていこうかな、ということで新規Dojoの追加方法を読んでみる
https://github.com/coderdojo-japan/coderdojo.jp/blob/master/docs/how-to-add-dojo.md

@yasulab
Copy link
Member Author

yasulab commented Apr 6, 2018

良いと思います! 👍 ✨

@yasulab
Copy link
Member Author

yasulab commented Apr 6, 2018

.oO(他のところでも結構評判が良いみたいなので、この動画頑張って収録・編集した甲斐がありました 💪 📹 ✨ )

収録・編集: YassLab
https://yasslab.jp/

@aokabin
Copy link
Contributor

aokabin commented Apr 7, 2018

db/dojos.yamlに書くだけで反映される...?のかな?

内容はさっくり2つのことが書いてあり、

  1. Dojo DBの追加手順
  2. 集計対象の追加

両方ともyamlファイルを書くだけで良さそうではあるかなぁ
一応1.に関しては、

yaml ファイルに各項目を追記したら、$ bundle exec rails dojos:update_db_by_yaml を実行してDBに新規Dojo情報を反映します。その後 $ bundle exec rails dojos:migrate_adding_id_to_yaml を実行します。

yamlファイルにidおよびorderが動的に更新されたことを確認できたら Add CoderDojo [Dojo名] でコミットをします。

という記述があるので、多少コマンドを入力する必要はありそうではあるが、あくまでもorderの値を自動紐付けする場合である(と読み取った)ので、手入力するなら必要はないかも...?

個人メモ

  1. db/xxx.yamlへの記述が、Rails標準の機能なのかgem使ってるのか調べてみる
  2. dojosコマンドが追加されてる、追加の仕方を知りたい

@yasulab
Copy link
Member Author

yasulab commented Apr 7, 2018

当時より便利タスクが増えたりドキュメントが用意されたりで少しコンテキストが異なりますが、全く同じタスクをアナグラさんにお願いした時の例が #259 になるので、こちらも参考になりそうです :)

@aokabin
Copy link
Contributor

aokabin commented Apr 11, 2018

情報をIDやorderの情報以外を書いたところで

  1. bundle exec rails dojos:update_db_by_yaml
  2. bundle exec rails dojos:migrate_adding_id_to_yaml
    を実行
    見た目には何か違いが出たわけではなかったので、成功したのか失敗したのかわからなかった。。。
    2はyamlファイルがアップデートされたので、わかったけど
    ログが出るわけではないので、何がどう変更されているのか、変更されていないのかがわからなかった。

ID2番という若い番号が割り振られたのも、何かミスがあったんじゃないかと不安...
(auto incrementとかを想像しがちなので、100個以上Dojoがあるのに2番は若すぎる)

心配していても埒があかないので、この時点でrails sして確認してみる

2018-04-11 15 43 15

おー、追加されてるされて...る...?
ふ、二つ追加されている...

rails c でコンソールから確認

[1] pry(main)> Dojo.where(name: "呉 (広島)")
  Dojo Load (0.9ms)  SELECT "dojos".* FROM "dojos" WHERE "dojos"."name" = $1  [["name", "呉 (広島)"]]
=> [#<Dojo:0x00007fee7d996948
  id: 2,
  name: "呉 (広島)",
  email: "",
  order: "342025",
  description: "呉市で毎月1回開催",
  logo: "/img/dojos/kure.jpg",
  url: "https://www.facebook.com/CoderDojo%E5%91%89-206840799906741/",
  tags: ["Webサイト"],
  created_at: Wed, 11 Apr 2018 15:31:50 JST +09:00,
  updated_at: Wed, 11 Apr 2018 15:31:50 JST +09:00,
  prefecture_id: 34>,
 #<Dojo:0x00007fee7d9967b8
  id: 1,
  name: "呉 (広島)",
  email: "",
  order: "342025",
  description: "呉市で毎月1回開催",
  logo: "/img/dojos/kure.jpg",
  url: "https://www.facebook.com/CoderDojo%E5%91%89-206840799906741/",
  tags: ["Webサイト"],
  created_at: Tue, 10 Apr 2018 18:26:35 JST +09:00,
  updated_at: Tue, 10 Apr 2018 18:26:35 JST +09:00,
  prefecture_id: 34>]

なんか怪しいなぁ
DB情報がなんか怪しいので、リセットしてみる

rails db:migrate:reset; rails db:seed; bin/setup

この時点で居る...!?

[2] pry(main)> Dojo.where(name: "呉 (広島)")
  Dojo Load (1.1ms)  SELECT "dojos".* FROM "dojos" WHERE "dojos"."name" = $1  [["name", "呉 (広島)"]]
=> [#<Dojo:0x00007fe0f9a57968
  id: 1,
  name: "呉 (広島)",
  email: "",
  order: "342025",
  description: "呉市で毎月1回開催",
  logo: "/img/dojos/kure.jpg",
  url: "https://www.facebook.com/CoderDojo%E5%91%89-206840799906741/",
  tags: ["Webサイト"],
  created_at: Wed, 11 Apr 2018 16:00:37 JST +09:00,
  updated_at: Wed, 11 Apr 2018 16:00:37 JST +09:00,
  prefecture_id: 34>]

yamlファイルに書いてあるからsetupの時に入っちゃうのかな?
yamlファイルから呉を削除して再度setupしてみる

すると、ちゃんと居なかった!正解!

焦って先にyamlファイルを書いて、その後setupとかをすると、同じDojoが複数個DBに登録されてしまうっぽい
(IDが1になってたのは未だ気になるけど...)

2018-04-11 16 40 52

ちゃんとできてる!

次は統計情報!

@aokabin aokabin mentioned this issue Apr 11, 2018
2 tasks
@aokabin
Copy link
Contributor

aokabin commented Apr 11, 2018

統計情報は、今回はFacebookの情報を登録するだけだったのですぐ終わった。
(Facebookだけが作業的に軽いのかどうかはわからない)

@yasulab
Copy link
Member Author

yasulab commented Apr 12, 2018

1コメントあたりの量が結構デカい印象があるので、もっと細かい単位で作業ログを書き込んだ方が周りの人たち (今回の件でいえば僕やアナグラさん) がサポートしやすそうですね ;)

僕の場合はいきなりドカッと情報量を書き込まれると、逆に読まない (正確には次にまとまった時間が取れるまで読まない → 次にまとまった時間が取れる頃にはもっと重要度の高い割り込みが発生している → 割り込みに対処し終わった頃には別の割り込みが... → 結果ずっと読まなくなる) 事が多いです >< 💦

細かく作業ログを書いて行くと作業の隙間時間に読めるので、周りの人たちとの連携もしやすいかなと思いました 😸

@yasulab
Copy link
Member Author

yasulab commented Apr 12, 2018

OSS Gate の動画でも kou さんは細かい単位でコメントしていったかなと思いますが、作業ログとしてコメントを残す場合は、具体的には次の Issue ぐらいの間隔/量で作業ログを次々と書き込んでいってもらえると助かります 👍
#259

@aokabin
Copy link
Contributor

aokabin commented Apr 13, 2018

意識してみます!ありがとうございます!

@yasulab
Copy link
Member Author

yasulab commented Apr 14, 2018

#321 で対応できたので、こちらの Issue 閉じますね ;) お疲れ様でした! ✨👏

1点、Description の次の部分だけ ✔︎ を付けずに残してありますが、もし追記した方が良い箇所などあれば別途 Issue or PR で対応できればなと考えています 📝 💨

  • 上記のドキュメントに加筆修正した方が良い箇所はある?

@yasulab yasulab closed this as completed Apr 14, 2018
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

No branches or pull requests

2 participants