ayaikg
commented
8 months ago MVPレビューありがとうございます！ いくつか補足説明と質問をさせていただきたいです。
- LINEログイン時に友達追加への導線（ずっと表示されてるのも違和感かもです）
LINEログイン時に友達追加の確認画面が表示されるように設定をしているのですが、自分でLINEログインをしたときは、なぜか表示されずにそのままログイン後のpathにリダイレクトされてしまったので、その後確認する方法が取れずに応急処置として、ログイン後のヘッダーに友達追加リンク設置していました。 そこでお聞きしたいのですが講師の方々で、アプリを確認していただいた時に、LINEログイン時に友達追加の確認画面は表示されましたでしょうか？
- ユーザー編集できるようにする
申し訳ございません。こちらのアドバイスの意味がよくわからず教えていただいてもよろしいでしょうか？ ユーザー編集ページは作成しており、nameカラム、avatarカラム、introductionカラムの編集ができる様にしております。 マイページに編集ボタンを設置しており、そこから編集ページにアクセスできる様にしております。 何か足りていないということでしょうか？
→申し訳ございません！こちらおそらく解決できたかと思います。LINEログインユーザーのemailがnilが複数あることでunique制約に引っかかってしまっていました。emailにvalidates :email, uniqueness: true, allow_nil: trueとallow_nil: trueを追加することでバリデーションをパスするように設定したところエラーが解消されました。
- ページネーションの表示する
申し訳ございません。こちらのアドバイスも意味がよくわからず教えていただいてもよろしいでしょうか？ ページネーションも導入しております。投稿一覧とお気に入り一覧と履歴ページに設定しております。投稿一覧とお気に入り一覧は10個まで、履歴ページには20個の表示を設定しております。こちらも何か足りていないということでしょうか？
- オートコンプリート検索はカテゴリ、アイテム一覧の方で使えるといいかもです
アイテム一覧と履歴ページにはオートコンプリートをすでに導入しております。現在、オートコンプリートはアイテム関連のページにのみ実装しています。 カテゴリー一覧は元々検索フォームを置いていましたが、
1.カテゴリーの変動がないためカテゴリー名での検索の必要性がないと思ったこと 2.カテゴリー一覧でアイテムの検索ができたら便利かと思いましたが、gemのancestryを使って親カテゴリーと子カテゴリーを作成しており、アイテムのcategory_idは子カテゴリーのidを格納していることから、検索キーをどのように設定すれば良いかわからない
以上の2点からカテゴリー一覧から検索フォームを削除していました。やはりカテゴリー一覧にも検索フォームを設置した方が良いですか？
- LINEログイン後PC画面からランキング一覧画面のリンクが隠れてしまうのがわかりにくいので修正する
ログイン前にヘッダーにある投稿ランキングのリンクのことでしょうか？こちらのことでしたら、未ログインユーザー専用のページとして作成したものになります。こちらは、ログイン後の投稿一覧の投稿の上位10つを表示する様にしています。 ですのでログイン後は表示はされません。ログイン後は通常の投稿一覧がランキングされた状態で表示される様に条件分岐をしております。
ログイン後も同様のページにアクセスできる様にした方が良いですか？
- マジックナンバー系は定数で定義して利用する
マジックナンバーとは例えば、enumで定義した数字をそのままコントローラで使っている様なことですか？
@posts = @q.result(distinct: true).eager_load([:item, :user]).preload(:likes) .where.not(items: { disposal_method: 0 }) .page(params[:page])
disposal_method: 0この0の部分をbeforeに変えるということでしょうか？
長文になってしまい、申し訳ございません。よろしくお願いいたします!
kerochelo
MVPリリースお疲れ様でした！
下記にUI/UXと技術面からのフィードバックを記載していますので、ご確認ください。
UI/UX
技術
上記の内容を確認しましたら、issueにタスクを落とし込んでもらって、スケジュール感を見積もっていただき共有していただいてもいいでしょうか？ 何か質問等あればコメントで投げていただければです！
以上よろしくお願いいたします。