お問い合わせ一覧と個別ページに「対応済」スタンプを実装 by riq0h · Pull Request #8535 · fjordllc/bootcamp

riq0h · 2025-04-06T05:35:21Z

Issue

#8465

概要

管理者画面のお問い合わせ一覧と個別ページに「対応済」スタンプを付けられるように実装しました。

変更確認方法

feature/add-completed-stampをローカルに取り込む。
　・git fetch origin feature/add-completed-stamp
　・git checkout feature/add-completed-stamp
foreman start -f Procfile.devで開発環境を立ち上げる。
　・ユーザー名komagata、パスワードtesttestでログインする。
任意のお問い合わせページにアクセスして「対応済にする」ボタンが表示されているか確認する。
「対応済にする」ボタンをクリックしてブラウザをリロード後、同ページに「対応済」スタンプが表示されているか確認する。（注：リアルタイムでスタンプが表示されないのは仕様です。ミーティングで了承済み。）
お問い合わせ一覧ページに移動して、「対応済にする」ボタンを押したページの項目に「対応済」スタンプが表示されているか確認する。

Screenshot

変更前

変更後

Summary by CodeRabbit

新機能
- 管理者画面の問い合わせ詳細ページに「対応済」スタンプと「確認者」表示を追加。
- 問い合わせに「対応完了」状態を設定できるボタンを追加し、状態変更が可能に。
- 問い合わせAPIに「対応完了」状態の更新エンドポイントを追加。
- 問い合わせ一覧で「対応完了」状態の絞り込みが可能に。
UIの改善
- 問い合わせ一覧・詳細で「対応済」や「確認者」などの視覚的インジケーターを表示。
パフォーマンス改善
- 問い合わせ一覧の表示速度を最適化。
バグ修正
- 一部関連データ取得時のエラーを防止。
内部処理の改善
- コードの可読性・保守性向上のためのリファクタリング。

riq0h · 2025-05-02T08:40:59Z

@thmz337 お疲れ様です！ご都合が合えばレビューを行っていただけませんでしょうか？ご検討よろしくお願いします🙇

thmz337 · 2025-05-02T14:55:47Z

ご指名ありがとうございます！
確認させていただきますね。

komagata · 2025-05-13T18:52:16Z

@thmz337 二週間ほど経過していますがこちらのレビューいかがでしょうか。
もし厳しければ他の方に代わりにレビューをお願いしても大丈夫です〜

thmz337 · 2025-05-14T13:37:55Z

@komagata

ご連絡ありがとうございます。
なかなか時間が取れずすいません。
今週の土日までにはと考えていますが、
もし難しそうであれば、他の方にご依頼させていただきます🙇

komagata · 2025-05-15T11:30:35Z

@thmz337

@riq0h さんとコミュニケーションを取り合って決めてみて下さい〜

riq0h · 2025-05-15T12:14:28Z

@thmz337 僕の方はそのスケジュール感でまったく問題ありません🙏 引き続きよろしくお願いします🙇

thmz337 · 2025-05-15T23:05:58Z

@riq0h
ありがとうございます🙇🫠
ご迷惑おかけしますが、よろしくお願いします。

thmz337 · 2025-05-18T06:00:28Z

app/controllers/articles_controller.rb

  before_action :set_article, only: %i[show edit update destroy]
  skip_before_action :require_active_user_login, raise: false, only: %i[index show]
  before_action :require_admin_or_mentor_login, except: %i[index show]
+  before_action -> { require_admin_or_mentor_login if params[:tag] }, only: %i[index]


こちらの変更点について、今回の修正と関係があるものなのでしょうか。

thmz337 · 2025-05-18T06:02:24Z

Gemfile.lock

多くのgemのバージョンが上がっていますが、問題なかったでしょうか。

thmz337 · 2025-05-18T06:09:46Z

app/controllers/api/inquiries_controller.rb

+# frozen_string_literal: true
+
+class API::InquiriesController < API::BaseController
+  def index


indexメソッドは使用されていますでしょうか。

thmz337 · 2025-05-18T06:12:23Z

db/schema.rb

  end

  create_table "categories", id: :serial, force: :cascade do |t|
-    t.string "name", limit: 255


「limit: 255」が軒並み消えていますが、問題ないでしょうか。

thmz337 · 2025-05-18T06:33:49Z

app/models/check.rb


  def receiver
-    checkable.user
+    checkable.respond_to?(:user) ? checkable.user : nil


checkableにuserメソッドが実装されていない場合、nilが返されるので、変更する必要がない気がします。

inquiry.rbにuserメソッドを実装するか、ここで明示的にnilを返す形にしないと以下のようにNoMethodErrorが発生してしまうため加えています。両方を実装していたのは余計でした。
NoMethodError (undefined method user' for #<Inquiry id: 657618196, name: "inquiry2", email: "inquiry2@example.com", body: "お問い合わせのテスト2です。", created_at: "2017-01-01 09:00:00.000000000 +0900", updated_at: "2025-05-18 18:46:10.031782079 +0900", action_completed: true>): 18:46:10 rails.1 | 18:46:10 rails.1 | app/models/check.rb:13:in receiver'
18:46:10 rails.1 | app/models/check_callbacks.rb:5:in after_create' 18:46:10 rails.1 | app/controllers/api/inquiries_controller.rb:8:in update'

確認したところ、おっしゃる通りでした。失礼しました。
こちらで問題ないかと思います。

thmz337 · 2025-05-18T06:35:43Z

app/models/check_callbacks.rb

 class CheckCallbacks
  def after_create(check)
-    if check.sender != check.receiver && check.checkable_type != 'Report'
+    if check.receiver && check.sender != check.receiver && check.checkable_type != 'Report' && check.checkable_type != 'Inquiry'


Inquiryにおいてはreceiverがnilを返すので、check.checkable_type != 'Inquiry'まで到達しないと考えます。
よって不要ではないでしょうか。

これがないと以下のようにバリデーションが失敗してしまうんですよね……。
GoodJob::Execution Update (0.5ms) UPDATE "good_jobs" SET "finished_at" = $1, "error" = $2, "updated_at" = $3 WHERE "good_jobs"."id" = $4 [["finished_at", "2025-05-18 09:38:01.802638"], ["error", "ActiveRecord::RecordInvalid: バリデーションに失敗しました: Userを入力してください"], ["updated_at", "2025-05-18 09:38:01.802908"], ["id", "d3dcf657-7967-45a5-b92d-c9066330beb4"]]

バリデーションが失敗するのはどのような操作を行なった時になりますでしょうか？

対応済みボタンを押した際です。

&& check.checkable_type != 'Inquiry'の箇所を削除して対応済みボタンを押したのですが、
問題なさそうです。

あれ？本当ですね……ではこの点はすぐに修正し、pushいたします。

@thmz337 pushしたのでご確認をお願いします🙇

@riq0h

修正ありがとうございます。
確認し問題ございませんでした。

thmz337 · 2025-05-18T06:41:01Z

app/models/inquiry.rb

+  scope :action_completed, -> { where(action_completed: true) }
+  scope :not_completed, -> { where(action_completed: false) }

+  def user


app/models/check.rbのrecieverメソッドで、こちらのuserメソッドを実装する・しないに関わらずnilを返すので、
不要な気がしますがいかがでしょうか。

thmz337 · 2025-05-18T06:47:10Z

@riq0h

お疲れ様です。レビュー遅くなってしまい、申し訳ございませんでした。
何点かコメントさせていただきましたので、ご確認お願いいたします。

riq0h · 2025-05-19T04:01:18Z

@thmz337 お疲れ様です！ご指摘いただいた不適切な箇所は削除し、そうでないと思われる箇所については返信を行いました🙏 お手隙の際に再度のレビューをよろしくお願いします🙇

riq0h force-pushed the feature/add-completed-stamp branch from 38117be to 31c2ce1 Compare April 6, 2025 09:19

riq0h requested a review from thmz337 May 2, 2025 08:38

machida assigned riq0h May 8, 2025

komagata force-pushed the main branch from 05531b1 to e7edcdb Compare May 8, 2025 06:00

thmz337 reviewed May 18, 2025

View reviewed changes

Gemfile.lock

Copy link
Copy Markdown

Contributor

thmz337 May 18, 2025

Choose a reason for hiding this comment

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

多くのgemのバージョンが上がっていますが、問題なかったでしょうか。

thmz337 reviewed May 18, 2025

View reviewed changes

thmz337 closed this May 18, 2025

thmz337 reopened this May 18, 2025

riq0h force-pushed the feature/add-completed-stamp branch 6 times, most recently from 0e7de8b to 0d46255 Compare May 19, 2025 02:35

riq0h requested a review from thmz337 May 22, 2025 12:57

riq0h force-pushed the feature/add-completed-stamp branch from 0d46255 to fcc10d3 Compare May 22, 2025 13:01

ryufuta mentioned this pull request Nov 18, 2025

管理者画面のお問い合わせ一覧、個別ページに「対応済」を付けられるようにしたい。 #8465

Closed

ryufuta mentioned this pull request Dec 2, 2025

ActionCompletedButtonComponentを再利用可能にする #9331

Merged

coderabbitai bot mentioned this pull request Jan 7, 2026

マイグレーションファイルをスキーマファイルと整合するように修復 #9485

Merged

Conversation

riq0h commented Apr 6, 2025 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Issue

概要

変更確認方法

Screenshot

変更前

変更後

Summary by CodeRabbit

Uh oh!

riq0h commented May 2, 2025

Uh oh!

thmz337 commented May 2, 2025

Uh oh!

komagata commented May 13, 2025

Uh oh!

thmz337 commented May 14, 2025

Uh oh!

komagata commented May 15, 2025

Uh oh!

riq0h commented May 15, 2025

Uh oh!

thmz337 commented May 15, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

thmz337 May 18, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

thmz337 commented May 18, 2025

Uh oh!

riq0h commented May 19, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

riq0h commented Apr 6, 2025 •

edited by coderabbitai bot

Loading

thmz337 May 18, 2025 •

edited

Loading