PR

Slack通知の関数化 #2558

kouho opened this at 2023-01-24T14:10:30; closed

Description

opened by kouho at 2023-01-24T14:10:30; updated 2023-02-03T10:33:14

概要

Slackの通知をゲーム内で関数にしました。以下の対応関連
https://git.dev.d-es.co.jp/gitbucket/jolly/jr-cs-issue/issues/273

テスト方法、確認した内容

確認用のページで関数を呼び出してSlackに通知が飛べばOK

サンプル

# jolly-yamadaに通知
S_SlackNotificationALL(SlackConst::COLOR_WARNING,"test_title","test_title2","test_message");

# jolly-yamada-developに通知
S_SlackNotificationDevelop(SlackConst::COLOR_WARNING,"test_title","test_title2","test_message");

TODO

  • スクリプトへの埋め込み

備考

  • developと全体通知の場合分けは引数を変えて呼び出すよりも違う関数にした方がわかりやすいかと思い分けて用意しました。
  • 色の定義をすることで返って入力内容が多くなってしまいますが直接warningやdangerなどを書くのは気になったため、定義をしてます。
  • 開発環境ではURLやchannelなどの情報はgit管理されていませんでしたが、リポジトリがprivateなのでgitで管理しました。

Comment

#25254 by kouho at 2023-01-24T14:10:59; updated 2023-01-24T14:10:59

@wada
レビューお願いいたします。

Comment

#25300 by kouho at 2023-01-26T12:26:29; updated 2023-01-26T12:26:29

@wada
迷いましたが、privateなリポジトリなのとdbのパスワードなどもdefineで既にgit管理しているので大丈夫かと判断しましたがまずい可能性がありそうでしょうか?

Comment

#25309 by wada at 2023-01-26T15:47:19; updated 2023-01-26T15:47:19

@kouho
確認しました。OKです。

wada add_label at 2023-01-26T15:47:21: 和田 LGTM
kouho merge at 2023-01-27T12:02:27: Slack通知の関数化
kouho close at 2023-01-27T12:02:27: Close
kouho refer at 2023-01-27T13:18:12: 2566:team_battle.php内バトルエラー発生時にSlack通知
kouho refer at 2023-02-03T10:33:14: 2581:【2/3】リリース・覚醒者選択ページ・Slack通知

commit comment

#157 by wada at 2023-01-26T12:15:14; lib/class/const/SlackConst.class.php / old:None new:13
32fba5247973f8bc689b1114abe9455337741d26

@kouho
URLとチャンネル名をgit管理して大丈夫ですか?

Commit range

base94907528faa3e81f972d6169508f62cb60fff1cb
head32fba5247973f8bc689b1114abe9455337741d26
base branchjolly/jr-web:develop
head branchjolly/jr-web:slack_function

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed2 filesView patch
statusfile
modifiedlib/SubFunc.inc
addedlib/class/const/SlackConst.class.php
Patch body is stored as a separate file to keep this page lightweight.

Review comment

#157 by wada at 2023-01-26T12:15:14; lib/class/const/SlackConst.class.php / old:None new:13
32fba5247973f8bc689b1114abe9455337741d26

@kouho
URLとチャンネル名をgit管理して大丈夫ですか?