Merge pull request #21 from devsecblueprint/DISCORD-6-Update-Discord-Bot-to-Use-Notification-Role

DISCORD-6 - Reverting to use Notifs role

Author: damienjburks

lambdas/discord-bot/src/handler.py

@@ -22,6 +22,7 @@
 NEWSLETTER_CHANNEL_NAME = os.environ["NEWSLETTER_CHANNEL_NAME"]
 JOB_BOARD_CHANNEL_NAME = os.environ["JOB_BOARD_CHANNEL_NAME"]
 ACTIVELY_HIRING_ROLE_ID = os.environ["ACTIVELY_HIRING_ROLE_ID"]
+NOTIFY_ROLE_ID = os.environ["NOTIFY_ROLE_ID"]
 
 # Logging Configuration
 logging.getLogger().setLevel(logging.INFO)
@@ -135,7 +136,7 @@ def process_video(body: str, channel_id: str):
     if payload is None:
         return "There was an issue processing the XML.", 500
 
-    message = f"Hey @everyone - Check out Damien's latest video - {payload['videoName']} {payload['videoUrl']}"
+    message = f"Hey <@&{NOTIFY_ROLE_ID}> - Check out Damien's latest video - {payload['videoName']} {payload['videoUrl']}"
     response = dynamodb_client.get_item(
         TableName=TABLE_ARN,
         Key={

terraform/core/main.tf

@@ -263,6 +263,7 @@ module "discord_bot" {
     "CONTENT_CORNER_CHANNEL_NAME" : "📹-content-corner"
     "JOB_BOARD_CHANNEL_NAME" : "🏢-job-board"
     "ACTIVELY_HIRING_ROLE_ID" : "1329874561818165309"
+    "NOTIFY_ROLE_ID" : "1338013529843826728"
   }
 
   create_permission    = true