search

coderdojo-japan / coderdojo.jp

☯️ CoderDojo Japan (@coderdojo-japan) official website developed by Ruby on Rails with @YassLab team. 💎
https://coderdojo.jp
Other
61 stars 102 forks source link

CoderDojo 鳥取からの coderdojo.jp 掲載依頼に対応する過程で、ドキュメントを整備したい #259

Closed yasulab closed 6 years ago

yasulab commented 6 years ago

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

対応例: https://github.com/coderdojo-japan/coderdojo.jp/commit/cf2d579ec961f2690f2a32220e0f8b17113412f0

cf. 作業メモの例 - OSS Gate

チェックリスト

作業の完了条件

AnaTofuZ commented 6 years ago

作業ログ

AnaTofuZ commented 6 years ago

be rspec spec/ テストが通る

追記したいので現在のDojoの数を確認

AnaTofuZ commented 6 years ago

みたところ　Dojoのidは実は考えなくてよく dojo.rakeに

    dojos.each do |dojo|
      d = Dojo.find_or_initialize_by(id: dojo['id'])

と書かれていて、なんとidが無いと自動でinitalizeされるのであった

冷静に見たら

d.name        = dojo['name']

と Dojo.find* から返ってきたオブジェクトがdに入っている為やはりidは必要だった

AnaTofuZ commented 6 years ago

Dojo の order フィールドは 全国地方公共団体コードの値が入っている ( #219 ) 全国地方公共団体コード一覧表 - Google Spreadsheet から　フィールドを取ってくる

スプレッドシートの情報と照らし合わせてyamlを追加する

- id: 124
  order: '312011'
  name: 鳥取
  prefecture_id: 31
  logo: "/img/docs/tottori.png"
  url: "https://www.facebook.com/events/162413071215576/"
  description: 鳥取県鳥取市で毎月開催
  tags:
  - Scratch
  - Ruby

またリンク先からDojoの画像をダウンロードし、所定の名前で　public/img/dojos に配置する。 yaml情報からデータを追加 be rails dojos:update_db_by_yaml

AnaTofuZ commented 6 years ago

created_at が正常にyamlに吐き出されていない為、今日の日付を記述し 念のため　be rails dojos:update_db_by_yaml 及び　be rails dojos:migrate_adding_id_to_yaml を行う。

sqliteを確認したところ created_at は当初raikeを起動した際に入ったと思われる2018-02-23 08:35:45.586225タイムスタンプが記録されている。

AnaTofuZ commented 6 years ago

ローカルで起動した所鳥取が2つ表示される状態になっていた。db操作が原因と思われるので be rails db:migrate:reset bin/setup しもう一度yamlから読ませる

また画像のリンク先をtypoしていたので修正した。 この状態で先程のrakeを2回実行し、ローカルで起動した所鳥取が表示された。 screen shot 2018-02-23 at 18 07 52

画像が寄っていたのでcropして修正

screen shot 2018-02-23 at 18 09 45

yasulab commented 6 years ago

画像が寄っていたのでcropして修正

👍 ✨

yasulab commented 6 years ago

みたところ　Dojoのidは実は考えなくてよく

ですね! 僕の場合は id 項目だけ削除したデータを yaml に追記して、その状態で $ be rails dojos:update_db_by_yaml ; be rails dojos:migrate_adding_id_to_yaml を実行しています。すると yaml ファイルに - id: xxx が差し込まれるので、それを commit & push (or PR) しています 🚀 ✨

AnaTofuZ commented 6 years ago

ありがとうございます!!

AnaTofuZ commented 6 years ago

見たところ鳥取はすでに次回のイベントがあるので db/dojos.yaml に次回のイベント情報を追加し be rails dojo_event_services:upsert 　した

AnaTofuZ commented 6 years ago

集計対象にも登録 なんですが、これはイベント情報をyamlに追加する以外に何か必要なのでしょうか? 🤔

yasulab commented 6 years ago

イベント管理サービスを db/dojo_event_services.yaml に追加する作業ですね ;) Rakeタスクの dojo_event_services:upsert 周りに詳しい実装があります 📜👀

AnaTofuZ commented 6 years ago

と思ったらすでにやっていました…ｗ

yasulab commented 6 years ago

コミット例: https://github.com/coderdojo-japan/coderdojo.jp/commit/52be3a2bb1d592fb0cc992a516957aecd92e0ed1

AnaTofuZ commented 6 years ago

あ、yamlファイルが違った…!!

AnaTofuZ commented 6 years ago

書き方(最後のfacebookURLが異なっていたので修正して再コミットしました)

yasulab commented 6 years ago

現在は Facebook / Doorkeeper / connpass の３つに対応していて、このいずれかの場合は自動集計ができるようになっています 🔧🤖💨 (スライド資料: https://speakerdeck.com/yasulab/coderdojo-japan-2018?slide=69 )

また、現状では人的リソースが足りないのであまり積極的に対応する必要はないのですが、Static に EventHistory を追加するインターフェイスもこちらに用意されています ;) https://github.com/coderdojo-japan/coderdojo.jp/issues/190

yasulab commented 6 years ago

.oO(下書きの段階から [WIP] hogehoge みたいなタイトルで PR を雑に作って、その branch 上で作業するとコードが共有できるので、WIP の段階からPRを出す習慣は身につけておくと良さそうです...!!)

例: https://github.com/coderdojo-japan/coderdojo.jp/pull/252

cf. How to write the perfect pull request - GitHub https://github.com/blog/1943-how-to-write-the-perfect-pull-request

Be explicit about when you want feedback, if the Pull Request is work in progress, say so. A prefix of “[WIP]” in the title is a simple, common pattern to indicate that state.

AnaTofuZ commented 6 years ago

ありがとうございます!!

12 #190 を見ているのですが,Facebookの group_id がどこから引けるか良く解ってないです。

PRを出す習慣忘れていました。(｀；ω；´)　PRベースでやっていきます!

yasulab commented 6 years ago

Facebookの group_id がどこから引けるか良く解ってないです。

Hint: [facebook group id search] 検索 (ポチッ

AnaTofuZ commented 6 years ago

ググり力が足りてなかった…! ありがとうございます!!

AnaTofuZ commented 6 years ago

作業は #261 で解決しました。ドキュメントの整備を行います

yasulab commented 6 years ago

お願いします! 😸 🙏