Skip to content
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.

Sign up for GitHub

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

Jump to bottom

相談部屋一覧を非vue化する #7447

Merged
merged 8 commits into from
Mar 9, 2024
Merged

相談部屋一覧を非vue化する #7447

merged 8 commits into from
Mar 9, 2024

Conversation

a-kuroki-gs
Copy link
Contributor

@a-kuroki-gs a-kuroki-gs commented Feb 27, 2024 •
edited
Loading

Issue

概要

相談部屋一覧の非Vue化対応を行いました。

  • 変更箇所
  • 見た目の変更箇所
    • JavaScriptを使わないように修正したため、以下の変更が入っております
    • 変更前
      • 検索欄に文字を入力した時点で検索される
    • 変更後
      • 検索欄に文字を入力し、Enterキーを押下した時点で検索される

変更確認方法

  1. feature/convert-talks-index-into-not-vueをローカルに取り込む
  2. 管理者でログインする
  3. http://localhost:3000/talks にアクセスして、以下を確認する
    • 相談部屋一覧がきちんと表示される
      • コメントがあれば、以下のように表示される
        image
    • ページネーションが機能している
    • 検索が機能している(半角英数字は3文字以上、それ以外は2文字以上)
      • 検索結果がない場合
        • 一致するものがない旨表示される
      • 検索結果がある場合
        • 一覧がきちんと表示される
    • ユーザーのステータスで絞り込みができる
    • タブが機能している
  4. http://localhost:3000/talks/action_uncompleted にアクセスして、以下を確認する
    • 未対応のものがない場合
      • 未対応のものがない旨表示される
    • 未対応のものがある場合
      • 未対応のものがきちんと表示される
    • 検索が機能している(半角英数字は3文字以上、それ以外は2文字以上)

Screenshot

Vueファイルの変更ですが、検索機能にのみ見た目の変更があるため以下に添付します。

変更前

  • 検索欄に文字を入力した時点で検索される

Image from Gyazo

変更後

  • 検索欄に文字を入力し、Enterキーを押下した時点で検索される

Image from Gyazo

@a-kuroki-gs
相談部屋一覧のVueファイルの内容をslim化する
904901b 
コントローラも修正し、app/controllers/talks_controller.rbに移動
@a-kuroki-gs
相談一覧画面のViewにおいて、ページネーションを実装
6fde3c4
@a-kuroki-gs
talks.vueを削除
018c986 
talk.vueは詳細画面でまだ使用しているので残す
不要になった設定も削除
@a-kuroki-gs
検索内容のバリデーションを設定
4e62cef
@a-kuroki-gs
検索した時のリクエスト先を現在地のパスに設定する
c821fd9 
未対応のタブを選択していても、検索後は全てのタブに遷移してしまっていたため
@a-kuroki-gs
Viewファイルの一部記述を修正
63a9fdb 
talksの存在を確認する箇所がエラーになっていたので、nil?からempty?に修正
.searched-talk-listの位置を正しくなるよう修正
それにともないelsifで書ける部分を修正
@a-kuroki-gs
その他テストの修正
8763cd6 
ローディング中のテストを削除
usericonのクラス表記を変更
検索する際にEnterキーを押す操作を追加
@a-kuroki-gs a-kuroki-gs self-assigned this Feb 27, 2024
@a-kuroki-gs a-kuroki-gs marked this pull request as ready for review February 28, 2024 01:11
@a-kuroki-gs
Copy link
Contributor Author

@natsuto6
お疲れ様です！
こちらのPRのレビューをお願いしてもよろしいでしょうか？
お時間厳しければ遠慮なくおっしゃってください🙌

ご確認よろしくお願いいたします🙏

@Takuya-Sakai91
Copy link
Contributor

@a-kuroki-gs
お疲れ様です！
急ぎでなければレビューさせていただきます🙌

Takuya-Sakai91
Takuya-Sakai91 approved these changes Mar 5, 2024
View reviewed changes
Copy link
Contributor

@Takuya-Sakai91 Takuya-Sakai91 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@a-kuroki-gs
お待たせしました🙇‍♂️
動作、コードともに特に問題ないと思いますので、Approveとさせていただきますー！

@a-kuroki-gs
Copy link
Contributor Author

@natsuto6
ご確認いただきありがとうございます🙌

@komagata
お疲れ様です！
メンバーにApproveをいただいたので、レビューをお願いいたします🙏

komagata
komagata approved these changes Mar 9, 2024
View reviewed changes
Copy link
Member

@komagata komagata left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

確認させて頂きました。OKです〜🙆‍♂️

@komagata komagata merged commit 54a104b into main Mar 9, 2024
4 checks passed
@komagata komagata deleted the feature/convert-talks-index-into-not-vue branch March 9, 2024 16:51
@github-actions github-actions bot mentioned this pull request Mar 9, 2024
Merged
29 tasks
@nishitatsu-dev nishitatsu-dev mentioned this pull request Apr 14, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Takuya-Sakai91 Takuya-Sakai91 Takuya-Sakai91 approved these changes

@komagata komagata komagata approved these changes

Assignees

@a-kuroki-gs a-kuroki-gs

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@a-kuroki-gs @Takuya-Sakai91 @komagata