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

管理者でログインしているときに相談部屋でコメントがあったときの通知メッセージの変更 #4107

Merged
merged 1 commit into from
Feb 9, 2022

Conversation

maeda-seina
Copy link
Contributor

@maeda-seina maeda-seina commented Feb 1, 2022

issue

概要

管理者でログインしているときに相談部屋でコメントがあったときの通知メッセージを変更

変更前

相談部屋でコメントした時の管理者通知(現役生(kimura)管理者(komagata))

  • サイト内通知

image

  • メール通知

image

他の管理者(machida)が相談部屋でコメントした時の管理者通知(現役生(kimura)管理者(komagata))

  • サイト内通知

image

  • メール通知

image

変更後

相談部屋でコメントした時の管理者通知(現役生(kimura)管理者(komagata))

  • サイト内通知

image

  • メール通知

image

他の管理者(machida)がコメントした時の管理者通知(現役生(kimura)管理者(komagata))

  • サイト内通知

image

  • メール通知

image

確認手順

1.任意のユーザーでログインする。
2.自分の相談部屋でコメントをする。
3.管理者(komagata)でログインする。
4.「〇〇さんの相談部屋で〇〇さんからコメントが届きました。」という通知メッセージでサイト内通知とメール通知が来ていることを確認する。(〇〇は先ほどのユーザーのlogin_name)
5.他の管理者(machida)でログインする。
6.先ほどのユーザーの相談部屋でコメントをする。
7.管理者(komagata)で再度ログインする。
8.「〇〇さんの相談部屋でmachidaさんからコメントが届きました。」という通知メッセージでサイト内通知とメール通知が来ていることを確認する。(〇〇は先ほどのユーザーのlogin_name)

@maeda-seina maeda-seina force-pushed the feature/change-the-talks-room-notification-message branch from 8d8b9d9 to 5052c2e Compare February 1, 2022 09:00
@maeda-seina maeda-seina force-pushed the feature/change-the-talks-room-notification-message branch from 5052c2e to 587c3d2 Compare February 6, 2022 09:35
@maeda-seina maeda-seina marked this pull request as ready for review February 6, 2022 12:57
@maeda-seina
Copy link
Contributor Author

@garammasala29 さん
お疲れ様です。
ご都合の良い時にこちらのレビューお願い致します🙏

Copy link
Contributor

@garammasala29 garammasala29 left a comment

Choose a reason for hiding this comment

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

@maeda-seina さん
お疲れ様です。レビュー依頼ありがとうございます!
自分の手元でも動作確認できました、LGTMです😄
maedaさんのおかげで、初めてletter_openerを使ってみることができました🙏

@maeda-seina
Copy link
Contributor Author

@garammasala29 さん
早々にレビューしていただき、ありがとうございます!!😊🙌

@komagata さん
お疲れ様です。
ご都合の良い時にこちらのレビューお願い致します🙏

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 76a25a6 into main Feb 9, 2022
@komagata komagata deleted the feature/change-the-talks-room-notification-message branch February 9, 2022 05:19
@github-actions github-actions bot mentioned this pull request Feb 9, 2022
54 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