fjordllc / bootcamp

プログラマー向けEラーニングシステム
https://bootcamp.fjord.jp
MIT License
286 stars 71 forks source link

イベント名がマークダウン記号から始まるのを禁止するバリデーションを設定 #8199

Closed hagiya0121 closed 11 hours ago

hagiya0121 commented 1 week ago

Issue

概要

変更確認方法

  1. bug/validate-event-nameをローカルに取り込む
    1. git fetch origin pull/8199/head:bug/validate-event-name
    2. git checkout bug/validate-event-name
  2. foreman start -f Procfile.devでローカルサーバーを立ち上げ
  3. 定期イベント作成ページに移動
  4. タイトル欄に# test(マークダウン記号から始まるタイトル)を入力してイベントを作成する
  5. タイトルには禁止されている記号が含まれていますというエラーメッセージが表示されることを確認

Screenshot

バリデーション設定後

image

hagiya0121 commented 1 week ago

@sugiwe お疲れ様です。 こちらのPRのレビューをお願いしたいです🙏 ご都合が悪いときは遠慮なくおっしゃってください🙇

sugiwe commented 1 week ago

@hagiya0121 お疲れ様です、レビューの件承知いたしました! 初レビューになりますので不慣れになるかもしれませんが是非やらせてください🙏 1週間ほどお時間いただければと思います、どうぞよろしくお願いいたします。

sugiwe commented 1 week ago

@hagiya0121 取り急ぎコード以外の部分になりますが、Assignees のところにご自身の設定をされていなかったようなので、設定しておくと良いかと思います👍

PR を作成したら自分をアサインする

hagiya0121 commented 6 days ago

@sugiwe 今気づいたのですがURLのPR番号を間違っていました。申し訳ありません。 レビューありがとうございます🙇抜け漏れ見つけていただいて助かります。 以下が変更点になります。ご確認お願いします🙏

sugiwe commented 2 days ago

@hagiya0121 反応が遅れてしまい申し訳ありません、修正を確認いたしました! マッチするMarkdownの内容も充実してテストもわかりやすくなり、良さそうに思います! メッセージの変更や違反箇所の表示もわかりやすくて良いと思いました◎

こちらからはApproveとさせていただきます!

hagiya0121 commented 2 days ago

@sugiwe お忙しいなかレビューありがとうございました🙇

hagiya0121 commented 2 days ago

@komagata メンバーのレビュー完了したので確認お願いします🙇