search

kaneko-joei / furima-31577

0 stars 0 forks source link

ユーザー管理機能 #3

Closed kaneko-joei closed 3 years ago

kaneko-joei commented 3 years ago

What ユーザー管理機能

why

下記のように修正 with_options presence: true do validates :nickname validates :password,format: { with:/\A(?=.?[a-z])(?=.?\d)[a-z\d]+\z/i } validates :birth_date

  with_options format: { with:/\A[ぁ-んァ-ン一-龠々]+\z/ } do
     validates :family_name
     validates :first_name
  end

  with_options format: { with: /\A[ァ-ヶー－]+\z/ }
     validates :family_name_kana
     validates :first_name_kana
  end

end

Why

/\A[ぁ-んァ-ン一-龠々]+\z/ こちらを利用しましょう 細かい点ですが不要な改行は削除 こちらは新規登録がうまくいかない異常系のテストのため 異常系の記述箇所に移動 英数字混合でない場合のパスワード include " エラーメッセージを入力を入力 passwordが英数字のみのとき登録できない passwordが数値のみのとき登録できない の2つ異常系テストを追加

Why

email,:passwordは既にdeviseで準備されていますので、削除 既にdeviseにて用意されているものですので、削除 パスワードは、半角英数字混合での入力が必須であること という要件が達成できるような、正規表現のバリデーションを追記 正規表現が重複しているバリデーションは、新たにwith_optionsを切り出してまとめる。 実装のためのご案内文ですので、削除 確実にpasswordに英数字が含まれるように指定 各テストのuser = FactoryBot.build(:user)は削除 テストコードは以下のようにcontextを使用して正常系と異常系とを分けて記述 emailに関しまして下記のテストも実施しましょう。 ・＠を含まなければ登録できないこと ・重複するemailだと登録できないこと passwordに関しまして、下記のテストコードの記述もお願いいたします。 ・パスワードは、6文字以上での入力が必須であること ・パスワードが必須であること ・半角数字のみでは登録できない ・全角では登録できないこと 名字・名前の各カラムに関しまして、下記のテストコードの記述もお願いいたします。 ・空だと登録できないこと ・半角文字だと登録できないこと フリガナカラムに関しまして、下記のテストコードの記述もお願いいたします。 ・空だと登録できないこと ・半角文字だと登録できないこと ・カタカナ以外の全角文字だと登録できないこと

why

コメントアウト、不要であれば削除 バリデーションは「with_options」を使用することで共通したオプションを付けることが可能 「〜しましょう」というコメントアウトは、実装案内用のコメントアウトです。 実装後は全て削除 実装案内用ですので、削除 すべての情報が登録できること 空では登録できないこと 漢字・平仮名・カタカナ以外では登録できないように正規表現を用いてバリデーションを設定 全角カタカナ以外では登録できないように、こちらもバリデーションを追記

Why

「ログイン画面」ができるキャプチャ動画をご提出 https://gyazo.com/dd6202c986cc8862974db9a15724cf1c

Why

・ユーザー登録の際、入力欄を全て空欄にし、全てのエラーメッセージが表示させれる。 https://gyazo.com/5ecb2c52d646371d074ca5ef5f32be35 ・ログインに成功する動画 https://gyazo.com/5dcddc9f1a340e5d64051bd81c42fd36

Why

deviseのパラメーターにnickname,email,password以外のカラムも記述 nickname以外のカラムに対してもバリデーションを記述 FactoryBotをつかってテストデータを作成 https://gyazo.com/ad6613ee50f3c6dbd6ea37a5f9eab6a2

What ユーザー管理機能

Why

ユーザーの姓、名、姓（フリガナ）、名（フリガナ）、生年月日を登録できるようにカラムを追加

What ユーザー管理機能

　新規登録に成功する動画 https://gyazo.com/f0970d237bd716797227fda0faf6a97a ・新規登録に失敗し、エラーが表示される動画 https://gyazo.com/b39507fbd2394e2ac3da8ddbd3df3bd4 ・ログインに成功する動画 https://gyazo.com/97869eb300c9bdfc75ee393a7cd35f83 ・ログインに失敗し、エラーが表示される動画 https://gyazo.com/c7f7da9d57af30cf43aa6cd3736a22ca ・ログアウトに成功する動画 https://gyazo.com/ea2a6455db298ff6bc8cbd3738eac2bb ・テスト結果の画像 https://gyazo.com/223eb6822fc0c86c6629beca2f8b3f64

What ユーザー管理機能

Why 新規登録ログインログアウトの実装

shodai-ito commented 3 years ago

レビュー依頼ありがとうございます！ こちらのプルリクエストですが、実際の画面や機能ができているか確認させていただきたいので、下記キャプチャ画像をご提出いただけますか？

・新規登録に成功する動画 ・新規登録に失敗し、エラーが表示される動画 ・ログインに成功する動画 ・ログインに失敗し、エラーが表示される動画 ・ログアウトに成功する動画 ・テスト結果の画像

「Gyazo　使い方」とgoogleで検索し、ツールによって生成されたURLをこのPRに貼っていただければ幸いです。 その後、再度レビュー依頼フォームにてご提出をお願いいたします。

yuri240 commented 3 years ago

レビュー依頼ありがとうございます。

前回のレビュアーからも指摘があったかと存じますが、 キャプチャ動画２点の再提出をお願いしております。

・ユーザー登録の際、入力欄を全て空欄にし、全てのエラーメッセージが表示させれる。
・ログインに成功する動画

こちらを拝見の上、再レビューさせていただきますので お手数ではありますがご提出をお願いいたします。

もしご不明点がございましたら 通話にてご質問いただけますと幸いでございます。

yuri240 commented 3 years ago

・ログインに成功する動画 https://gyazo.com/5dcddc9f1a340e5d64051bd81c42fd36

ご提出いただきました「ログイン画面」ですが、 deviseのデフォルトで用意されているものを使用されているようです。

カリキュラムにて配布されたビューが表示されるように 修正をしていただき、こちらが完了しましたら 再度正常にログインができるキャプチャ動画をご提出お願いいたします。

ご不明点がございましたら 通話での質問を活用いただけますと幸いでございます。