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

add tottri dojo #261

Merged
merged 3 commits into from Feb 26, 2018
Merged

add tottri dojo #261

merged 3 commits into from Feb 26, 2018

Conversation

@AnaTofuZ
Copy link
Member

@AnaTofuZ AnaTofuZ commented Feb 23, 2018

#259で鳥取からcoderdojo.jpに掲載依頼が来ていた為対応した
今のPRで行ったことは以下の通りです

  • ローカルにrailsの環境を構築
  • 鳥取の情報をyamlに追加
  • 鳥取の画像をcropして設置
  • イベント情報を追加
  • Facebook_idを検索し、yamlに追加する

ドキュメントは別途行います

#259

\#259で鳥取からcoderdojo.jpに掲載依頼が来ていた為対応した
今のPRで行ったことは以下の通りです

- ローカルにrailsの環境を構築
- 鳥取の情報をyamlに追加
- 鳥取の画像をcropして設置
- イベント情報を追加

ドキュメントは別途行います

#259
name: 鳥取
prefecture_id: 31
logo: "/img/dojos/tottori.png"
url: https://www.facebook.com/events/162413071215576/

This comment has been minimized.

@yasulab

yasulab Feb 23, 2018
Member

いい感じですね!

依頼されたURLについてですが、イベント単体へのリンクだと情報が古くなりがちなので (イベントが開催されるたびに更新する必要がてできてしまうので)、イベントページよりは Facebook ページ https://www.facebook.com/CoderDojoTottori/ の方が、ユーザーはいつどのタイミングでアクセスしても最新の情報にたどり着けて良さそうですね ;)

依頼されたURLとは違いますが、今回は https://www.facebook.com/CoderDojoTottori/ の方を登録しちゃいましょう 📝 💨

個別イベントのid,urlをyamlに記述しており,毎回書き換える必要が出てしまった為
dojo_event_services.yamlで記述していた鳥取のGroupID,urlを修正した

facebook_id は[Lookup-ID.com](https://lookup-id.com/#) を利用した
@AnaTofuZ AnaTofuZ changed the title [WIP]add tottri dojo add tottri dojo Feb 26, 2018
@AnaTofuZ
Copy link
Member Author

@AnaTofuZ AnaTofuZ commented Feb 26, 2018

コミットメッセージがわかりづらいものとなってしまっていた…。気をつけます

@yasulab
Copy link
Member

@yasulab yasulab commented Feb 26, 2018

コミットメッセージがわかりづらいものとなってしまっていた…。気をつけます

@AnaTofuZ お、であれば rebase -i してコミットメッセージを直してみてはいかがでしょう? 🤔(まだマージしてないのでコミットメッセージは直せそう)

urlが個別イベントに向いていた為こちらも修正した
@AnaTofuZ AnaTofuZ force-pushed the AnaTofuZ:add_totori_dojo branch to 1df7124 Feb 26, 2018
@AnaTofuZ
Copy link
Member Author

@AnaTofuZ AnaTofuZ commented Feb 26, 2018

git rebase -i 26d52022145a4d63b7ded81cc098c044c30a6108 を行いコミットメッセージを修正した後に
git push --force origin add_totori_dojo してゴリっと修正しました

@yasulab
Copy link
Member

@yasulab yasulab commented Feb 26, 2018

初PR! 🎉 良さそうなのでマージしますね😸

@yasulab yasulab merged commit e2d4954 into coderdojo-japan:master Feb 26, 2018
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@AnaTofuZ AnaTofuZ deleted the AnaTofuZ:add_totori_dojo branch Feb 26, 2018
@AnaTofuZ
Copy link
Member Author

@AnaTofuZ AnaTofuZ commented Feb 26, 2018

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.