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

newspaperの引数をハッシュでの受け渡しに統一 1/2 #7175

Merged
merged 1 commit into from
Jan 24, 2024

Conversation

2525nicole
Copy link
Contributor

@2525nicole 2525nicole commented Jan 3, 2024

Issue

概要

newspaperを呼び出す際に引数をハッシュで受け渡しするように統一しました。
※対応範囲が広いため、2つのPRに分けています。

もう1件のPRは以下
newspaperの引数をハッシュでの受け渡しに統一 2/2

変更確認方法

  1. feature/unify-newspaper-arguments-with-hash_1/2をローカルに取り込む
  2. 以下の「変更確認前の下準備」を行う
  3. foreman start -f Procfile.devを実行し、ローカル環境を立ち上げる
  4. 「動作確認」をそれぞれ試し、newspaperの引数をハッシュでの受け渡しに変更する前と変わらない処理が行われることを確認する

変更確認前の下準備

キャッシュ削除確認のための準備

app/models/learning_cache_destroyer.rbcallメソッドの1行目にputs "LearningCacheDestroyer#call"を追記する

動作確認

チェックリスト ※必要に応じてご利用いただければと思います🙇‍♀️

  • Learningのキャッシュ削除処理
  • 休会復帰時の処理
  • 提出物関連の処理
  • 相談部屋のコメント投稿時の処理

Learningのキャッシュ削除処理

  1. muryouでログイン
  2. 未着手のプラクティスDebianをインストールする
    ページに行き、ステータスを着手に変更する
  3. ターミナルに"LearningCacheDestroyer#call"が出力されていることを確認する
  4. komagataでログインする
  5. http://localhost:3000/users/746986380ページに行き、muryouを削除する
  6. ターミナルに"LearningCacheDestroyer#call"が出力されていることを確認する

休会復帰時の処理

  1. 休会からの復帰ページにアクセスし、休会中ユーザーkyuukaiの復帰手続きを行う
    ※メールアドレスとパスワードはこちら

  2. http://localhost:3000/letter_opener/にアクセスし、komagata宛に「[FBC] kyuukaiさんが休会から復帰しました。」というメール通知が飛んでいること、メール本文内の「kyuukaiさんのページへ」をクリックするとkyuukaiのプロフィールページに遷移することを確認する

  3. komagataでログインし、右上のベルマーク通知で、「kyuukaiさんが休会から復帰しました!」と表示されること、クリックするとkyuukaiのプロフィールページに遷移することを確認する


提出物関連の処理

  1. mentormentaroでログインする

  2. Terminalの基礎を覚えるプラクティスにアクセスし、「Watch」を押下する
    スクリーンショット 2024-01-02 14.21.20.png

  3. kensyuでログインする

  4. Terminalの基礎を覚えるプラクティスにアクセスし、提出物を作成する

  5. 提出後、提出物がWatch中になっていることを確認する

  6. http://localhost:3000/letter_opener/にアクセスし、「[FBC] kensyuさんが「Terminalの基礎を覚える」の提出物を提出しました。」というメール通知が以下2名に飛んでいること、メール本文内の「提出物へ」をクリックすると4で作成した提出物ページに遷移することを確認する

  • アドバイザーのsenpai
  • プラクティスWatch中のmentormentaro
  1. senpaiでログインし、ページ右上のベルマーク通知で、「kensyuさんが「Terminalの基礎を覚える」の提出物を提出しました。」と表示されること、通知をクリックすると4で作成した提出物に遷移することを確認する
  2. mentormentaroでログインし、ページ右上のベルマーク通知で、「kensyuさんが「Terminalの基礎を覚える」の提出物を提出しました。」と表示されること、通知をクリックすると4で作成した提出物に遷移することを確認する
  3. 再び`kensyu`でログインする
  4. Terminalの基礎を覚えるプラクティスのステータスを「着手」に変更する
  5. 4で作成した課題を「内容修正」→下書き保存し、プラクティスのステータスが「着手」のままであることを確認する
  6. Terminalの基礎を覚えるプラクティスのステータスを「未着手」に変更する
  7. 11で下書き保存した課題を「内容修正」→再び下書き保存し、プラクティスのステータスが「着手」になることを確認する
  8. Terminalの基礎を覚えるプラクティスのステータスを再び「未着手」に変更する
  9. Debianをインストールするプラクティスのステータスを「着手」に変更する
  10. 13で下書き保存した Terminalの基礎を覚えるプラクティスの課題を再度下書き保存し、プラクティスのステータスが「未着手」のままであることを確認する
  11. 16で下書き保存したTerminalの基礎を覚えるプラクティスの課題を提出する
  12. komagataでログインし、17で提出された課題ページに飛び「課題を確認する」を押下する
  13. kensyuでログインし直し、Terminalの基礎を覚える
    プラクティスのステータスが「修了」になっていることを確認する

相談部屋のコメント投稿時の処理

  1. komagataでログインする
  2. kimuraさんの相談部屋にアクセスし、コメントする
  3. http://localhost:3000/letter_opener/にアクセスし、kimura宛に「[FBC] 相談部屋でkomagataさんからコメントがありました。」という件名のメール通知が飛んでいること、メール本文内の「コメントへ」をクリックすると2で投稿したコメントに遷移することを確認する
  4. kimuraでログインし、ページ右上のベルマーク通知で、「相談部屋でkomagataさんからコメントがありました。」と表示されること、通知をクリックすると2で投稿したコメントに遷移することを確認する
  5. kimurakimuraさんの相談部屋でコメントをする
  6. http://localhost:3000/letter_opener/にアクセスし、komagata宛に「[FBC] kimuraさんの相談部屋でkimuraさんからコメントが届きました。」という件名のメール通知が飛んでいること、メール本文内の「コメントへ」をクリックすると2で投稿したコメントに遷移することを確認する
  7. komagataでログインし、ページ右上のベルマーク通知で、「kimuraさんの相談部屋でkimuraさんからコメントが届きました。」と表示されること、通知をクリックすると5で投稿したコメントに遷移することを確認する

Screenshot

画面上の変更点はありません

@2525nicole 2525nicole self-assigned this Jan 3, 2024
@2525nicole 2525nicole marked this pull request as ready for review January 3, 2024 07:45
@2525nicole 2525nicole requested a review from niikz January 3, 2024 07:54
@2525nicole
Copy link
Contributor Author

@niikz
※年始に失礼いたします><
おつかれさまです!
今年もよろしくお願いいたします🙇‍♀️🎍
急ぎではありませんので、ご都合のよろしいタイミングでレビューをお願いできますでしょうか🙏

@2525nicole 2525nicole changed the title newspaperの引数をハッシュでの受け渡しに統一 newspaperの引数をハッシュでの受け渡しに統一 1/2 Jan 3, 2024
@2525nicole
Copy link
Contributor Author

@niikz
たびたび恐れ入ります><!
先日ご依頼をさせていただいたこちらいかがでしょうか🙏
ご対応が難しい場合は遠慮なくご教示いただけますと幸いです🙇‍♀️
お手数ですが、ご確認の程よろしくお願いいたします🍀

@niikz
Copy link
Contributor

niikz commented Jan 9, 2024

@2525nicole
お疲れ様です。確認が遅くなってしまい申し訳ありません…🙏
1週間以内に対応いたします🙇 よろしくお願いします!

@2525nicole
Copy link
Contributor Author

2525nicole commented Jan 9, 2024

@niikz
おつかれさまです✨とんでもないです!!
お忙しいところご確認をいただきありがとうございます🙇‍♀️!
お手数をおかけいたしますが、どうぞよろしくお願いいたします🙏

@niikz
Copy link
Contributor

niikz commented Jan 16, 2024

@2525nicole
遅くなりすみません…🙇‍♀️コンフリクトでpullできないためご確認いただけますでしょうか?
お手数ですがよろしくお願いします🙏

@2525nicole 2525nicole force-pushed the feature/unify-newspaper-arguments-with-hash_1/2 branch from 524cb2b to b623aca Compare January 17, 2024 03:24
@2525nicole
Copy link
Contributor Author

@niikz
とんでもないです〜!!お忙しいところありがとうございます🙇‍♀️
最新のmainをrebaseしましたので、ご確認をいただけますでしょうか🙏
たびたびお手数をおかけいたしますがよろしくお願いいたします🍀

Copy link
Contributor

@niikz niikz left a comment

Choose a reason for hiding this comment

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

@2525nicole
ありがとうございます🙏
確認できました!私からはOKです〜🙆‍♀️

@2525nicole
Copy link
Contributor Author

@niikz
おつかれさまです!
お忙しい中お引き受けいただき、またご確認ありがとうございました🙇‍♀️
引き続きよろしくお願いいたします😊🍀

@komagata
おつかれさまです!
チームメンバーの方にApproveをいただきましたので、レビューをお願いできますでしょうか🙇‍♀️

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 31eb0a9 into main Jan 24, 2024
7 checks passed
@komagata komagata deleted the feature/unify-newspaper-arguments-with-hash_1/2 branch January 24, 2024 05:54
@github-actions github-actions bot mentioned this pull request Jan 24, 2024
28 tasks
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.

3 participants