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

お知らせの初回公開時に通知が発行されるように修正 #5662

Merged
merged 2 commits into from
Oct 21, 2022

Conversation

hikarook94
Copy link
Contributor

概要

#5591
こちらのPRの修正です。確認方法などはこちらのPRをご確認ください。

事象

お知らせをWIPで保存→公開というフローで操作するとお知らせの通知が発行されないという事象をステージング環境で確認しました。

修正内容

updateアクション実行時にpublished_atnilから変更された時(=初回公開時)に通知を発行する処理を追加しました。

@hikarook94 hikarook94 marked this pull request as ready for review October 20, 2022 08:29
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.

こちら再発を防ぐためのテストがあるといいかもです〜

@hikarook94
Copy link
Contributor Author

@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.

テストの追加、大変ありがたいです〜!

@komagata komagata merged commit 5ae818b into main Oct 21, 2022
@komagata komagata deleted the bug/add-announce-notifier-at-the-first-publishing branch October 21, 2022 14:52
@github-actions github-actions bot mentioned this pull request Oct 21, 2022
10 tasks
@hikarook94
Copy link
Contributor Author

@komagata @machida
お疲れ様です!
こちらのPRについて、メンター権限が必要なため本番環境で動作確認をお願い出来ますでしょうか?

@komagata komagata changed the title 初回公開時に通知が発行されるように修正 お知らせの初回公開時に通知が発行されるように修正 Oct 25, 2022
@komagata
Copy link
Member

@hikarook94 本番環境で動作することを確認しました。

@hikarook94
Copy link
Contributor Author

@komagata
ご確認ありがとうございます!

@hikarook94 hikarook94 self-assigned this Oct 26, 2022
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.

2 participants