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

FAQにカテゴリを追加する #7093

Merged
merged 101 commits into from
Nov 18, 2024
Merged

FAQにカテゴリを追加する #7093

merged 101 commits into from
Nov 18, 2024
+1,699 −472

Conversation

sochi419
Copy link
Contributor

@sochi419 sochi419 commented Dec 1, 2023 •
edited
Loading

Issue

概要

こちらのPRでyumaさんがFAQのCRUD機能を実装した。今回のPRで、yumaさんが実装したFAQのCRUD機能にカテゴリ機能を追加した。
以下の実装を行った。

変更確認方法

  1. feature/add-category-for-faqsをローカルに取り込む
  2. rails db:setupを実行する
  3. http://localhost:3000/faqにアクセス
  4. カテゴリタブを選択して、カテゴリ毎に表示が切り替わることを確認する。(カテゴリタブは下記画像の赤で囲った箇所です。デザインが入ったら画像を差し替える予定です。)
image
  1. http://localhost:3000/admin/faqs/newにアクセス
  2. 「質問」「回答」に任意な文字を入力する。カテゴリーは任意のカテゴリーを選択する。その後「登録する」ボタンを選択する。
  3. http://localhost:3000/admin/faqsにアクセス
  4. 下にスクロールして6.で作成したFAQが表示されていることを確認する。表示されているカテゴリと、6.で選択したカテゴリが一致していることを確認する。

Screenshot

変更前

image image

変更後

  • http://localhost:3000/faq
    ※ 画像はカテゴリタブのデザインがまだ入っていない状態なので、デザインが入ったら画像を差し替えます。
image image

@sochi419 sochi419 force-pushed the feature/add-faq-features-in-admin-page branch from a7ff054 to 1e14fcb Compare December 5, 2023 06:21
@sochi419 sochi419 self-assigned this Dec 5, 2023
@sochi419 sochi419 force-pushed the feature/add-category-for-faqs branch from 2dd7cdc to 1431811 Compare December 5, 2023 06:22
@sochi419 sochi419 force-pushed the feature/add-faq-features-in-admin-page branch from 1e14fcb to 1d0a287 Compare December 12, 2023 02:24
@sochi419 sochi419 force-pushed the feature/add-category-for-faqs branch from 1431811 to d90e8da Compare December 12, 2023 02:25
@sochi419 sochi419 force-pushed the feature/add-faq-features-in-admin-page branch from 1d0a287 to 7b512f1 Compare January 9, 2024 00:21
@sochi419 sochi419 force-pushed the feature/add-category-for-faqs branch from d90e8da to dbf6199 Compare January 9, 2024 00:23
@sochi419 sochi419 force-pushed the feature/add-faq-features-in-admin-page branch from 7b512f1 to 5d6b16c Compare January 17, 2024 01:52
@sochi419 sochi419 force-pushed the feature/add-category-for-faqs branch from dbf6199 to 5417a94 Compare January 17, 2024 01:52
@sochi419 sochi419 force-pushed the feature/add-faq-features-in-admin-page branch from 5d6b16c to c686c50 Compare February 4, 2024 14:41
@sochi419 sochi419 force-pushed the feature/add-category-for-faqs branch from 5417a94 to 33b2edd Compare February 4, 2024 14:42
@sochi419 sochi419 force-pushed the feature/add-faq-features-in-admin-page branch from c686c50 to 9d0c3a1 Compare February 6, 2024 05:44
@sochi419 sochi419 force-pushed the feature/add-category-for-faqs branch 4 times, most recently from 1793342 to 19b3170 Compare February 12, 2024 04:50
@sochi419 sochi419 force-pushed the feature/add-category-for-faqs branch 2 times, most recently from a08f4e1 to 9c629fa Compare February 27, 2024 01:59
@sochi419 sochi419 force-pushed the feature/add-category-for-faqs branch 3 times, most recently from b75b91a to 6772716 Compare March 4, 2024 08:13
@sochi419 sochi419 changed the base branch from feature/add-faq-features-in-admin-page to main March 5, 2024 02:18
@sochi419 sochi419 force-pushed the feature/add-category-for-faqs branch from b5ad47e to 17e975e Compare March 5, 2024 11:41
@sochi419 sochi419 marked this pull request as ready for review March 5, 2024 15:00
@sochi419
Copy link
Contributor Author

sochi419 commented Mar 5, 2024

@omochiumaiumai
お疲れ様です！
こちらのPRのレビューをお願いさせていただきたくてご連絡させていただきました🙏

ご対応可能なようでしたら、お手すきの際にでもご確認いただけますと幸いです。
よろしくお願いいたします。

@omochiumaiumai
Copy link
Contributor

@sochi419
お疲れ様です！
申し訳ありませんが少々立て込んでおりレビューをするのに十分な時間が確保できないため、他の方にお願いしていただけるとありがたいです🙇‍♂お役に立てずすみません😭

@sochi419
Copy link
Contributor Author

sochi419 commented Mar 7, 2024

@omochiumaiumai
連絡ありがとうございます！
承知しました！

@sochi419 sochi419 removed the request for review from omochiumaiumai March 7, 2024 00:43
@sochi419 sochi419 force-pushed the feature/add-category-for-faqs branch from 17e975e to a879736 Compare March 26, 2024 05:04
sochi419 and others added 22 commits November 15, 2024 14:46
@sochi419 @machida
デザイン修正影響で通らなくなったテストを修正
ac682c7
@sochi419 @machida
admin側のFAQのshowを削除
4b0858d
@sochi419 @machida
FAQページのカテゴリタブをeach文で簡略化
855db3f
@sochi419 @machida
lintエラーを解消
c6c51c6
@sochi419 @machida
モバイル表示されるselectタグの選択肢を日本語化、カテゴリを選択しても「全て」になったままだった不具合を修正
84b519b
@sochi419 @machida
セレクトタブでカテゴリ選択すると、即座にページが切り替わるよう修正
8a509fb
@sochi419 @machida
enum削除、faq_categoryモデルを使った処理に変更
fa010f8
@sochi419 @machida
enum削除に合わせてtestを修正
ab1dc78
@sochi419 @machida
rubocop指摘点を修正
0264bbd
@sochi419 @machida
モバイル表示の不具合修正、lintエラー修正
f8821c8
@sochi419 @machida
データ取得時に並び順を指定する処理を追加
84e72e7
@sochi419 @machida
モデル、テーブル名の変更
670c85f
@machida
新デザインを適用し、とりあえず動くようにした
10a30b5 
動かなくなったFAQの順番移動を動くようにした
@machida
FAQデザイン更新
431f183 
👮

👮

faqs categoryをfaq categoryに修正

FaqをFAQに修正

FaqをFAQに修正

FaqをFAQに修正

デザイン変更に伴うテスト変更’

デザイン変更に伴うseed変更’

並べ替えを動くようにした

不要なカラムを削除

👮

ATCH リクエストで faq_category_id パラメータが送信されていなかったのを修正

リファクタ

testでのカテゴリーの指定方法を修正
@komagata @machida
FAQカテゴリーの管理画面を追加
0e778c2
@komagata @machida
不要になったテストの削除
9072ca4
@komagata @machida
FAQカテゴリーに関するテストを追加
dd23a7e
@komagata @machida
使っていないroutesを削除
3717493
@komagata @machida
FAQ関連の失敗しているテストを修正
f61c0bd
@machida
FAQにデザインを入れ、初期データのカテゴリを修正、管理画面のタブを整理した
3481ba6
@komagata @machida
UIの変更にテストを追従させた
de66a6e
@komagata @machida
fixtureに不要な文字が混入していたのを修正
a4833bc
@machida machida force-pushed the feature/add-category-for-faqs branch from 64e8e96 to a4833bc Compare November 15, 2024 05:47
@machida
Copy link
Member

machida commented Nov 15, 2024

@komagata

@komagata あ、すいません！表側の部分でデザイン漏れがありそうな気がしました。今出先なのですが、戻り次第すぐ確認して連絡します🙇‍♂️

この件問題なしでした。勘違いでした。

@komagata
Copy link
Member

@machida 問題なさそうなのでマージしちゃいますね。

@komagata komagata merged commit af2499a into main Nov 18, 2024
1 check passed
@komagata komagata deleted the feature/add-category-for-faqs branch November 18, 2024 04:21
@github-actions github-actions bot mentioned this pull request Nov 18, 2024
Open
8 tasks
@machida
Copy link
Member

machida commented Nov 18, 2024

@komagata ありがとうございます！

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@machida machida machida left review comments

@komagata komagata komagata left review comments

@masyuko0222 masyuko0222 masyuko0222 approved these changes

Assignees

@komagata komagata

@machida machida

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@sochi419 @omochiumaiumai @masyuko0222 @komagata @machida @yuma-matsui