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.

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

FBC内の分報の削除・編集機能の追加 #8362

Draft
wants to merge 27 commits into
base: main
Choose a base branch
from

Conversation

SuzukiShuntarou
Copy link
Contributor

@SuzukiShuntarou SuzukiShuntarou commented Feb 21, 2025

Issue

概要

  • フィヨルドブートキャンプ内の分報について、編集と削除機能の追加。
    分報の新規作成、表示は別のPRで実装済。(本番環境では分報タブを非表示にしている)
  • 作成者とadmin権限を持つユーザのみが分報の編集、削除が可能。

変更確認方法

  1. feature/update-and-delete-about-micro-reportsをローカルに取り込む
    1. git fetch origin feature/update-and-delete-about-micro-reports
    2. git checkout feature/update-and-delete-about-micro-reports
  2. rails db:seedを実行して動作確認用の分報の初期データを追加する。
  3. foreman start -f Procfile.devローカル環境を立ち上げる。
  4. ユーザー名 hajime パスワード testtest でログインする。
  5. マイプロフィールからプロフィール画面を表示する。
  6. タブの分報(28)をクリックする。
  7. 分報の最新のページ(2ページ目)が表示される。
  8. 各分報の【内容修正】ボタンを押すと【コメント】と【プレビュー】タブが表示され、それぞれに登録された初期データの内容が表示されていることを確認する。
  9. 任意の分報の【内容修正】ボタンを押し、【コメント】タブから内容を変更した後に【プレビュー】タブに切り替えると表示内容が切り替わることを確認する。
  10. 【保存する】ボタンを押すと変更した内容が分報に表示されることを確認する。
  11. 任意の分報の【内容修正】ボタンを押し、【コメント】タブから内容を変更した後に【キャンセル】ボタンを押すと、修正内容が反映されないことを確認する。
  12. 1ページ目に移動し、表示される任意の分報の【削除する】を押し確認画面でOKを選択すると、選択した分報が削除されるので以下を確認する。
    1. 『分報を削除しました。』というフラッシュメッセージが表示されること
    2. 選択した分報が削除されていること
    3. タブの分報が【分報(27)】になっていること
    4. 元のページ(1ページ目)の分報が表示されること(ページネーションの選択が『1』となっていること)
  13. 2ページ目に移動し、表示される任意の分報の【削除する】を押し確認画面でOKを選択すると、選択した分報が削除されるので以下を確認する。
    1. 選択した分報が削除されていること
    2. タブの分報が【分報(26)】になっていること
    3. 元のページ(2ページ目)の分報が表示されること(ページネーションの選択が『2』となっていること)
  14. ログアウトしてからユーザー名 komagata パスワード testtest でログインする。(adminユーザ)
  15. 検索覧でhajimeを入力して検索し、プロフィールを表示する。
  16. タブの分報(26)をクリックする。
  17. 分報に【内容修正】と【削除する】が表示されていることを確認する。
  18. ログアウトしてからユーザー名 mentormentaro パスワード testtest でログインする。(非adminユーザ)
  19. 検索覧でhajimeを入力して検索し、プロフィールを表示する。
  20. タブの分報(26)をクリックする。
  21. 分報に【内容修正】と【削除する】が表示されていないことを確認する。

Screenshot

変更前

  • 新規作成した分報の表示のみ、【内容修正】と【削除】は表示されない。
    前

変更後

  • 作成者、または、adminユーザの場合、【内容修正】と【削除】の追加
    image

  • 【内容修正】を押すと、【コメント】【プレビュー】タブの表示
    image
    image

  • 作成者でない、かつ、非adminユーザは【内容修正】と【削除】が表示されない
    image

@SuzukiShuntarou SuzukiShuntarou force-pushed the feature/update-and-delete-about-micro-reports branch from 49cf350 to 5486cf9 Compare February 21, 2025 05:39
@SuzukiShuntarou SuzukiShuntarou force-pushed the feature/update-and-delete-about-micro-reports branch from 5486cf9 to cf620a7 Compare February 21, 2025 05:54
@SuzukiShuntarou
Copy link
Contributor Author

@machida

  • お疲れ様です!分報の編集画面のボタンにつきまして、デザインをお願いいたします。discordの質問部屋にてボタンのデザインを変更されると伺いました。以下の【保存する】と【キャンセル】になります。
    image
  • app/components/users/micro_reports/micro_report_component.html.slimが対象の編集画面となります。

@machida
Copy link
Member

machida commented Feb 27, 2025

@SuzukiShuntarou 一旦、一覧の方のデザインを更新しましたー。デザインを更新してJSが動かなくなってしまったので、動くようにしていただきたいです。

そのときに、既存のclassを使ってJSを発火させるのではなく、js-xxxx という js プリフィクスを付けた新しいクラスを使って発火させるようにして、デザインとjsを分離するようにしていただきたいです🙏

@SuzukiShuntarou
Copy link
Contributor Author

@machida

  • デザインの対応ありがとうございます!js プリフィクスを付けた新しいクラスを追加しそちらを用いて発火させるように変更いたしました。
  • 編集・削除について追加したシステムテストについてはデザインに合わせて変更をしました。(ローカルで通っていることを確認済です。)元々あった一覧表示のシステムテストについてはデザインが途中であったためまだ落ちている状況です。

お手数をおかけしますが、再度ご対応をどうぞよろしくお願いいたします。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants