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

質問を更新した際にDiscordに通知が走らないようにした #6990

Merged

Conversation

omochiumaiumai
Copy link
Contributor

@omochiumaiumai omochiumaiumai commented Oct 23, 2023

Issue

概要

質問投稿時と質問更新時に重複してdiscordに通知が走っていたため、質問投稿時とwip公開時のみ通知が走るようにしました。

変更確認方法

  1. feature/discord question notification only for new postsブランチをローカルに取り込む
  2. Develop環境でのDiscord通知の確認方法 を参考にテスト用のサーバーを用意する
  3. http://localhost:3000/questions で質問を作成
  4. discordで質問投稿の通知を受け取る
  5. 作成した質問の本文を編集して更新する
  6. discordに通知が走らないことを確認する
  7. 作成した質問の本文を編集してwipにする
  8. wipを公開し、discordで質問投稿の通知を受け取る

Screenshot

変更前

image

変更後

image

@omochiumaiumai
Copy link
Contributor Author

@sochi419
お疲れ様です!
こちらのレビューをお願いしたいのですが、ご都合はいかがでしょうか?
お手すきの際にご返信をいただければ幸いです🙇‍♂

@omochiumaiumai omochiumaiumai marked this pull request as ready for review October 24, 2023 10:14
@sochi419
Copy link
Contributor

@omochiumaiumai

承知しました!
1週間以内に対応させていただきます🙏

あと、CIのテストが落ちているみたいなので、確認よろしくお願いします〜

@omochiumaiumai
Copy link
Contributor Author

@sochi419
ありがとうございます!

あと、CIのテストが落ちているみたいなので、確認よろしくお願いします〜

依頼前に気づくべきだったのですが、修正の影響で別のテストが通らなくなってしまったようなので現在処理を見直しています💦
修正後に改めてレビューを依頼したほうがよいでしょうか?

@omochiumaiumai
Copy link
Contributor Author

@sochi419
お疲れ様です!
修正に時間がかかり、ご迷惑をおかけしてしまうため一度レビューの依頼を白紙に戻させて頂きたく存じます🙇‍♂💦
こちらからお願いしたのにもかかわらず、大変申し訳ございません。

@sochi419
Copy link
Contributor

@omochiumaiumai

承知しました〜
すみません、前回のコメントに返信していなくて申し訳ありませんでした🙏
(返信していたと勘違いしていました😅)

@omochiumaiumai omochiumaiumai removed the request for review from sochi419 October 27, 2023 06:01
@omochiumaiumai
Copy link
Contributor Author

@Sasaco-JM
お疲れ様です!
もしよろしければこちらのレビューをお願いしたいのですが、ご都合はいかがでしょうか?
お手すきの際にご返信をいただければ幸いです🙇‍♂

@Sasaco-JM
Copy link
Contributor

@omochiumaiumai
お疲れ様です。
返信が遅くなり大変申し訳ありません。
今から1週間以内の対応でも問題ありませんでしょうか。

@omochiumaiumai
Copy link
Contributor Author

@Sasaco-JM
お疲れ様です!ご確認いただきありがとうございます。
1週間以内の対応で問題ございません。よろしくお願いいたします🙇‍♂

Copy link
Contributor

@Sasaco-JM Sasaco-JM left a comment

Choose a reason for hiding this comment

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

@omochiumaiumai
お疲れ様です。
質問作成時、WIP→公開時のみ通知がされ、編集時は通知されないことを確認しました。
問題ないと思いますので、Approveします 🙇

@omochiumaiumai
Copy link
Contributor Author

@Sasaco-JM
お疲れ様です!
ご確認いただきありがとうございました🙇‍♂

@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 523d926 into main Dec 5, 2023
5 checks passed
@komagata komagata deleted the feature/discord_question_notification_only_for_new_posts branch December 5, 2023 02:59
@github-actions github-actions bot mentioned this pull request Dec 5, 2023
11 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.

4 participants