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

定期イベントでメンション付きのコメントをした場合、メールと通知のメッセージが不適切 #6365

Open
wata00913 opened this issue Mar 24, 2023 · 9 comments
Assignees

Comments

@wata00913
Copy link
Contributor

wata00913 commented Mar 24, 2023

概要

定期イベントにて、メンション付きのコメントをすると、メールと通知のメッセージが不適切です。

原因

メッセージを作成する際に、Mentionertarget_of_commentメソッドを呼び出します。引数のcommentable_classにマッチしたメッセージを返す処理ですが、RegularEventに紐づくメッセージが存在しないためだと考えられます。

バグの原因と思われるソースコード

再現手順

  1. regular_events/{id}にアクセスする
  2. ログインユーザー以外のユーザーにメンションを付けてコメントを投稿する
  3. メンションしたユーザーでログインをする

通知の確認

  1. notificationsにアクセスし、通知のタイトルを確認する

メールの確認

  1. letter_openerにアクセスし、Subjectを確認する

期待される振る舞い

通知とメールのメッセージが以下のような形式であること。

  • 通知
定期イベント「`タイトル名`」へのコメントで`ユーザー名`さんからメンションがきました。
  • メール
[FBC] 定期イベント「`タイトル名`」へのコメントで`ユーザー名`さんからメンションがありました。

スクリーンショット

  • 通知

image

  • メール

image

@github-actions
Copy link

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

@github-actions github-actions bot added the stale label Jun 20, 2023
@wata00913 wata00913 removed the stale label Jun 20, 2023
@github-actions
Copy link

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

@komagata
Copy link
Member

📝 e6i24vi さんが休会されたのでアサインを外させていただきました。

@github-actions
Copy link

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

@github-actions github-actions bot added the stale label Oct 22, 2023
@komagata komagata removed the stale label Oct 23, 2023
Copy link

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

Copy link

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

@github-actions github-actions bot added the stale label Mar 24, 2024
@peno022 peno022 removed the stale label Mar 24, 2024
Copy link

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

@komagata komagata removed the stale label May 28, 2024
Copy link

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

@github-actions github-actions bot added the stale label Jul 28, 2024
@komagata komagata removed the stale label Aug 2, 2024
@komagata komagata moved this to やる in bootcamp Aug 21, 2024
Copy link

github-actions bot commented Oct 2, 2024

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: レビュー中
Development

No branches or pull requests

6 participants