質問投稿時の通知に質問のタイトルが表示されるようにした #4350
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
tksmasaki
force-pushed
the
feature/show-question-title-in-notification
branch
from
8c2e5d3 to
4120dc2
Compare
tksmasaki
commented
Mar 5, 2022
| @@ -103,6 +106,9 @@ notification_create_page: | |||
| user: hatsuno | |||
| sender: komagata | |||
| message: "komagataさんがDocsにBootcampの作業のページを投稿しました。" | |||
| link: "/pages/<%= ActiveRecord::FixtureSet.identify(:page4) %>" | |||
| read: false | |||
There was a problem hiding this comment.
Issue と別件ですが、hatsuno でログイン後にこの通知をクリックした際、リンクが存在せずにエラーになっていたため、修正しました。
There was a problem hiding this comment.
修正してくださってありがとうございます！
こちらもhatsunoでログインして通知をクリックして動作確認をしたところ、大丈夫でした〜🙆♀️
tksmasaki
commented
Mar 5, 2022
| @@ -16,15 +15,12 @@ class Notification::QuestionsTest < ApplicationSystemTestCase | |||
| fill_in('question[title]', with: 'メンターに質問！！') | |||
| fill_in('question[description]', with: '通知行ってますか？') | |||
| end | |||
| first('.select2-selection--single').click | |||
| find('li', text: '[Mac OS X] OS X Mountain Lionをクリーンインストールする').click | |||
There was a problem hiding this comment.
デフォルトでもプラクティスの項目は選択されているため、記述を削除しました。
（質問作成の動作確認がテストの目的ではないため、不要と判断しました）
|
@Saki-htr |
Saki-htr
approved these changes
Mar 13, 2022
There was a problem hiding this comment.
@tksmasaki さん
お待たせして申し訳ありません🙇♀️動作確認したところOKでした!
メール通知の形式も同じでした。
テストも丁寧に書かれていて良いと思います😊
LGTMです〜
| @@ -103,6 +106,9 @@ notification_create_page: | |||
| user: hatsuno | |||
| sender: komagata | |||
| message: "komagataさんがDocsにBootcampの作業のページを投稿しました。" | |||
| link: "/pages/<%= ActiveRecord::FixtureSet.identify(:page4) %>" | |||
| read: false | |||
There was a problem hiding this comment.
修正してくださってありがとうございます！
こちらもhatsunoでログインして通知をクリックして動作確認をしたところ、大丈夫でした〜🙆♀️
komagata
approved these changes
Mar 14, 2022
| @@ -103,6 +106,9 @@ notification_create_page: | |||
| user: hatsuno | |||
| sender: komagata | |||
| message: "komagataさんがDocsにBootcampの作業のページを投稿しました。" | |||
| link: "/pages/<%= ActiveRecord::FixtureSet.identify(:page4) %>" | |||
32 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue
概要
質問投稿時に送られる通知の件名に、質問のタイトルが表示されるようにしました。
=> 文言のフォーマット：
XXXさんから質問「質問のタイトル」が投稿されました。変更前
サイト内通知
メール通知
変更後
サイト内通知
メール通知
ローカル環境での確認方法
サイト内通知
feature/show-question-title-in-notificationブランチを起動mentor? == trueの他のユーザーでログインし直し、通知の中にXXXさんから質問「質問のタイトル」が投稿されました。の形式で通知が届いていることを確認するmentormentaro、komagataなどメール通知
feature/show-question-title-in-notificationブランチを起動[bootcamp] XXXさんから質問「質問のタイトル」が投稿されました。の形式であることを確認する