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

入館申請のQRコードの送信先を選択できるように修正しました #214

Merged
merged 9 commits into from
Jun 11, 2024

Conversation

shigerix
Copy link
Contributor

@shigerix shigerix commented May 16, 2024

やりたいこと

入館に必要なQRコードの送信先を選べるようにする

やったこと

  • メッセージの送信先を設定ファイルの内容によって変更できるようにしました。
  • CHANNELの場合
    • チャンネルのみにメッセージを通知する
    • 投稿されたメッセージのスレッドに詳細情報を通知する
  • DMの場合
    • ワークフローの申請者のDMのみにメッセージを通知する
    • DMに投稿されたメッセージのスレッドに詳細情報を通知する
  • BOTH の場合
    • チャンネルとDMどちらともにメッセージを投稿する
    • DMに投稿されたメッセージのみに詳細情報を通知する

@shigerix shigerix self-assigned this May 16, 2024
@shigerix shigerix requested a review from yoyogix May 29, 2024 10:32
@shigerix shigerix marked this pull request as ready for review May 30, 2024 03:04
@shigerix shigerix requested a review from a team as a code owner May 30, 2024 03:04
@shigerix
Copy link
Contributor Author

@yoyogix お手漉きの際にレビューをお願いします!

@shigerix shigerix changed the title [WIP]入館申請のQRコードの送信先を選択できるように修正する 入館申請のQRコードの送信先を選択できるように修正する May 30, 2024
Copy link
Contributor

@yoyogix yoyogix left a comment

Choose a reason for hiding this comment

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

細かな部分ですがコメントさせていただきました 🙏

@@ -30,6 +31,52 @@
EMAIL_BODY
end

describe "#post" do
# SEND_MODE が CHANNEL の場合
# post_public_message が1回呼ばれること
Copy link
Contributor

Choose a reason for hiding this comment

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

コメントの消し忘れですかね?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

消し忘れですね🙏削除しました!

@shigerix shigerix requested a review from yoyogix May 30, 2024 08:08
@shigerix shigerix changed the title 入館申請のQRコードの送信先を選択できるように修正する 入館申請のQRコードの送信先を選択できるように修正しました Jun 7, 2024
@shigerix shigerix requested a review from yoyogix June 7, 2024 07:52
@shigerix
Copy link
Contributor Author

shigerix commented Jun 7, 2024

@yoyogix メッセージの追加を行ったのでもう1度レビューをお願いします!

@shigerix shigerix merged commit a973da3 into kufu:main Jun 11, 2024
2 checks passed
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