-
-
Notifications
You must be signed in to change notification settings - Fork 110
🆕 Add CoderDojo 八女 in 福岡県 #1767
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
Conversation
db/dojo_event_services.yml
Outdated
| ### 以下、Dojo 情報まとめ ### | ||
| ### (dojos.yml の追加順) ### | ||
|
|
||
| # 八女 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@nacchan99 コチラ、他のコメントのパターンと揃えてもらえると助かります! 🙇
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
失礼しました🙌ご指摘ありがとうざいます!
5eae005 にて修正しました!
@nacchan99 さんが渡した URL が、記載されている渡し方と違うからだと思います! 🙇 💦 - bundle exec bin/c-search https://coderdojo-yame.connpass.com/event/
+ bundle exec bin/c-search https://coderdojo-yame.connpass.com/ |
- 150行から49行に削減(約67%削減) - validate_and_classify_url と fetch_group_by_subdomain メソッドを削除 - 例外処理、バリデーション、タイムアウト処理をすべて削除 - HOW_TO_USE_MESSAGE 定数でメッセージを一元化 - 数字とイベントURLの処理を統合(or 条件使用) - エラーが起きたら Ruby の標準エラーをそのまま表示する設計に変更 - ハイフン付きサブドメイン対応: \w+ → [\w-]+ - 正規表現にアンカー追加: ^ → \A (セキュリティ向上) - Ruby 2.7+ の it 記法を使用(番号付きブロックパラメータ) - 変数名を統一: res/data → response/result - インデントを揃えて可読性向上 シンプルな3パターンの処理のみ: 1. 数字のみ → イベントID 2. /event/数字 を含む → イベントURL 3. https://XXX.connpass.com → グループURL
渡し方が異なっていた場合、実行時に気づけるように修正しました! 🛠💨✨ 789e55f |
|
@yasulab コメントと早速の修正対応ありがとうございます!🙏✨
bundle exec bin/c-search https://coderdojo-yame.connpass.com/
=> エラーが発生しました: SSL_connect returned=1 errno=0 peeraddr=3.169.5.85:443 state=error: certificate verify failed (unable to get certificate CRL)各イベント管理サービスの group_id の取得方法 にある、
ここの「イベントページ」= https://coderdojo-yame.connpass.com/event/ だと勘違いしておりました🙌💦 私の環境に依存する問題のようなので、手元で解消していきます! |
やったこと
補足(connpass の group_id 取得について)
各イベント管理サービスの group_id の取得方法 に記載されているコマンドを実行したところ、以下のような結果になり、 group_id の取得ができませんでした。
そのため、connpass のグループページに埋め込まれている
/series/<id>/のパスを HTML から直接抽出する方法に切り替えました。さらに広い範囲で確認した場合も、複数箇所に
17099が含まれています◎https://connpass.com/series/17099/ にアクセスすると、CoderDojo 八女のトップページへ遷移することも確認済みです!
(例で提示されている CoderDojo 戸部でも、 https://connpass.com/series/5072/ にアクセスすると、CoderDojo 戸部のトップページへ遷移されます🙌)