Theaninova/openstapps
1
0
Fork 0
mirror of https://gitlab.com/openstapps/openstapps.git synced 2026-01-11 12:12:55 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

feat: make slack notification optional

Browse Source
This commit is contained in:
Rainer Killinger
2020-10-30 18:04:35 +01:00
parent ccb1b56a0f
commit 84921ad5ce
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/tasks/remind.ts
View File

@@ -51,7 +51,9 @@ export async function remind(api: Api): Promise<void> {
Logger.info(`Found ${mergeRequests.length} open merge requests.`); Logger.info(`Found ${mergeRequests.length} open merge requests.`);
// instantiate slack client // instantiate slack client
const client = new WebClient(process.env.SLACK_API_TOKEN); const client = typeof process.env.SLACK_API_TOKEN !== 'undefined' ?
new WebClient(process.env.SLACK_API_TOKEN) :
undefined;
// get members of main group // get members of main group
const members = await api.getMembers(MembershipScope.GROUPS, GROUPS[0]); const members = await api.getMembers(MembershipScope.GROUPS, GROUPS[0]);
@@ -136,7 +138,7 @@ export async function remind(api: Api): Promise<void> {
.join(', '); .join(', ');
// send message to slack // send message to slack
await client.chat.postMessage({ await client?.chat.postMessage({
channel: SLACK_CHANNEL, channel: SLACK_CHANNEL,
text: `Merge request '${mergeRequest.title}' needs more approvals! See ${mergeRequest.web_url}!`, text: `Merge request '${mergeRequest.title}' needs more approvals! See ${mergeRequest.web_url}!`,
}); });
@@ -152,7 +154,7 @@ export async function remind(api: Api): Promise<void> {
Logger.log(`Merge request '${mergeRequest.title}' is ready to be merged!`); Logger.log(`Merge request '${mergeRequest.title}' is ready to be merged!`);
// send message to slack // send message to slack
await client.chat.postMessage({ await client?.chat.postMessage({
channel: SLACK_CHANNEL, channel: SLACK_CHANNEL,
text: `Merge request '${mergeRequest.title}' is ready to be merged! See ${mergeRequest.web_url}!`, text: `Merge request '${mergeRequest.title}' is ready to be merged! See ${mergeRequest.web_url}!`,
}); });