search

kazukiSaga / anaba

0 stars 0 forks source link

MVPリリースレビュー依頼 #41

Closed kazukiSaga closed 1 month ago

kazukiSaga commented 1 month ago

各種情報

サービスの概要とMVPで検証する事柄：

READMEレビュー時から変更した機能

スポット編集後にスポット詳細画面に遷移予定であったが、スポット一覧画面へ遷移する設計としました。 卒業認定までの開発スケジュール見積もり面談にて講師と確認をとり、以下の機能をMVP時の実装から本リリース時の実装へ変更しました。

kazukiSaga commented 1 month ago

前回レビュー依頼したissueをclosedからopenにすることができなかったため、issueを別に作り直しました。 MVPリリースが今日までという、急な提出で申し訳ありません。再度レビューをお願いします。

前回までのコメント

・MVPリリースのレビューをお願いします。

・Closed状態になってますね。（Closed状態のものをレビューすることは一般的に無いので対応の上、再提出をお願いいたします。）

Tsuchiya2 commented 1 month ago

https://github.com/user-attachments/assets/8415881b-a1d5-48a2-8d96-78a51dfa67a3

ログイン操作後、ヘッダーの項目が変わらないのですが、これはちゃんとログインできているのか分からなかったです。（ログイン・未ログインでの違いがあればテキストにて説明をお願いします。）

kazukiSaga commented 1 month ago

レビューありがとうございます。 確認したところ、何らかが原因で現状、新規登録・ログイン後にページのリロードを行わないとヘッダーの変更が反映されないようになっているようです。 新規登録、もしくはログイン後リロードしていただければ、機能の確認自体はできるかと思われます。 検索機能はスポット一覧ページから検索できるようになっております。

kazukiSaga commented 1 month ago

お疲れ様です。テキストにサービスの概要とMVPで検証する事柄についてが抜けていたので追記しました。 MVPリリースが今日までという、急な提出で申し訳ありません。再度レビューをお願いします。

Tsuchiya2 commented 1 month ago

機能確認が取れました。LGTMです！ 引き続き本リリースに向けて開発頑張ってください。

以下、LGTMを出した方々に共通してお伝えしている内容になります。 （Webアプリを就職活動で活用される方向けの内容になります）

===== 本リリースまでに以下も取り組む ===== □ 独自ドメイン反映 -> DNSやSSLの対応も必要になるのでアピールになり、URLを書類等に記載するときの見栄えも良くなります。 □ OGP（静的 or 動的） -> SNS等でシェアするときの見栄えが良くなります。Webアプリであればどんなアプリでも取り入れられるので対応しておきましょう。 □ RSpec（モデルテスト+システムテスト or モデルテスト+リクエストテスト） -> 開発における品質の意識や開発効率の意識の高さをアピールできます。Request Specをメインに最低限各機能に正常にアクセスできるか程度はテストで確認できるようにしておきましょう。 □ CI（GitHubActions） -> 実際に開発現場でも使われており、アプリ外で技術のアピールが出来るものです。RSpecと合わせて実装を行いましょう。 □ Rubocopを使ってのリファクタリング -> Lintチェックツールは開発現場で広く使われています。コードのLintを整えておくことはチームで開発することを意識するうえでとても大切になるので取り入れておきましょう。

他のプログラミング学習者のWebアプリと異なることをアピールするために、よりインパクトのある機能を複数盛り込むことが重要になってきます。 □ SNSログイン（Google or LINE） -> ほとんどのアプリに取り入れられてユーザー体験の向上や技術のアピールにつながるので取り入れましょう。 □ 検索機能のアップデート（オートコンプリート、マルチ検索等） -> 特にオートコンプリートは検索体験の向上だけではなくJS系の技術も取り入れていることをアピール出来るので積極的に取り入れましょう。

ユーザー体験という面では以下の項目も大切です。 □ オートログイン -> ユーザー登録後に再度ログイン操作を行わせないことでユーザー体験を向上させられます。 □ 使い方説明がある -> 初めて訪れたユーザーに呈して使い方の説明があることで体験を向上させられます。 □ レスポンシブ対応 -> いずれのデバイスでも綺麗にサービスが活用できることで体験が良くなります。 □ 利用規約・プライバシーポリシーを未ログイン状態でも確認できる -> 利用規約・プライバシーポリシーがあることで、ちゃんとしたサービスという印象を与えられます。 □ ローディングアニメーション（処理が重い箇所） -> 処理が重い箇所に何の対応も無いとユーザーが正しく操作できたのか困惑してしまいます。そうした際はローディングアニメーションを表示させることで体験が向上します。

kazukiSaga commented 1 month ago

お忙しい中 レビューありがとうございました。