Skip to content

岡南の情報とロゴ更新+イベント収集サイト追加 #686

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

Merged
merged 2 commits into from
Jan 8, 2020

Conversation

chicaco
Copy link
Contributor

@chicaco chicaco commented Jan 5, 2020

背景

CoderDojo 岡南 の Dojo 情報の変更依頼への対応

cf. #678

やりたいこと

岡南の情報とロゴを修正し、イベント収集サイトを追加する

このPRでやること

  • db/dojos.yaml : 岡南のDojo名と説明の修正
  • db/dojo_event_services.yaml : 岡南のイベント情報収集サイト( https://coderdojo-konan.doorkeeper.jp/ )を追加
  • public/img/dojos/okayama.png : ロゴファイルの入れ替え
  • rake dojos:update_db_by_yaml の動作確認
  • rake dojo_event_services:upsert の動作確認
  • rake statistics:aggregation の動作確認
  • (追加) db/dojos.yaml : 岡南のURL修正

やらなかったこと

特になし

困っていること

特になし

@chicaco chicaco self-assigned this Jan 5, 2020
@yasulab yasulab temporarily deployed to coderdojo-japan-pr-686 January 5, 2020 03:58 Inactive
@chicaco chicaco requested a review from Yuppymam January 5, 2020 09:34
@yum45f
Copy link

yum45f commented Jan 6, 2020

@chicaco すみません! 岡南道場 の URL も変更をお願いします... これだけ issue に前のやつ書いてませんでした... わかり辛くて申し訳ないです...

url: https://www.coderdojo-konan.jp/

@chicaco
Copy link
Contributor Author

chicaco commented Jan 7, 2020

岡南道場 の URL も変更をお願いします

こちらこそ見落としてしまい、お手数をお掛けしました。

Copy link
Contributor

@Yuppymam Yuppymam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

変更確認しました🙆‍♀️✨
イベントサービスの変更もダブル載せで良さそうです👌
マージして大丈夫そうです✨
スクリーンショット 2020-01-08 18 41 13

@chicaco
Copy link
Contributor Author

chicaco commented Jan 8, 2020

📝 デプロイ後にやることメモ

  • 岡南(dojo_id: 81)の統計情報再収集、2019年7月以降

@chicaco chicaco merged commit a8f22a0 into master Jan 8, 2020
@chicaco chicaco deleted the mod_konan_dojo branch January 8, 2020 14:29
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

Successfully merging this pull request may close these issues.

4 participants