🆕 Add CoderDojo 砧 in 東京都 #1738
Merged
🆕 Add CoderDojo 砧 in 東京都 #1738
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
yasulab
reviewed
Aug 12, 2025
db/dojos.yaml
Outdated
| prefecture_id: 13 | ||
| logo: "/img/dojos/default.webp" | ||
| url: https://connpass.com/event/363462/ | ||
| description: 世田谷区で毎月1回開催 |
There was a problem hiding this comment.
@nacchan99 ここは他の説明文に合わせて 毎月開催 としてください! そのまま採用するのではなく、なるべく既存の説明文の文章パターンを合わせてもらえると助かります...!! 🙏 ✨
|
@nacchan99 こちらは 1Password for Team の内容をご確認いただければちゃんと動くかなと思います...!! 🙇
|
- id を本番環境の連番に合わせて修正 - ConnpassイベントのURLを更新 - 説明文を既存の文章パターンに統一 - dojo_event_services.yaml にコメント追加
db/dojo_event_services.yaml
Outdated
| @@ -27,6 +27,12 @@ | |||
| ### 以下、Dojo 情報まとめ ### | |||
| ### (dojos.yaml の追加順) ### | |||
|
|
|||
| # 砧（東京都世田谷区）- connpass の個人アカウントのため登録できない。グループアカウントにする必要がある。（グループ化できたら dojos.yamlのurlも変更すると良さそう） | |||
There was a problem hiding this comment.
めちゃくちゃ細かいのですが、後でやること・やりたいことは TODO として記載するとわかりやすそうです👍
- # 砧（東京都世田谷区）- connpass の個人アカウントのため登録できない。グループアカウントにする必要がある。（グループ化できたら dojos.yamlのurlも変更すると良さそう）
+ # 砧（東京都世田谷区）- connpass の個人アカウントのため登録できない。グループアカウントにする必要がある。
+ TODO: グループ化されたら dojos.yamlのurlも合わせて更新There was a problem hiding this comment.
ご指摘ありがとうございます！
1e69da7にて変更しました✨
rakuda-san-desu
approved these changes
Aug 12, 2025
|
マージします🚀 |
|
本番環境でも確認できました！ |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
やったこと
メモID=342（播磨科学公園都市 in 兵庫県）が欠番になっているため、CIテストの Dojo validate id sequence が失敗中。#1723 がマージされれば解決しそう。