From fa1dd3fcde7222ce3af19cad5654512f68b1b00b Mon Sep 17 00:00:00 2001
From: Joey Hines <joey@ahines.net>
Date: Fri, 10 Jul 2020 09:29:54 -0500
Subject: [PATCH] Small fixes

+ Creating "ping_roles" more idiomatically
+ Updated readme
---
 README.md          |  2 ++
 src/discord/mod.rs | 11 ++++-------
 2 files changed, 6 insertions(+), 7 deletions(-)

diff --git a/README.md b/README.md
index d2fee77..bc07568 100644
--- a/README.md
+++ b/README.md
@@ -35,6 +35,8 @@ prefix = "~"
 event_channel = 0
 # List of roles that can use the bot
 event_roles = [0]
+# List of roles to ping when a new event is posted
+ping_roles = [0]
 # Timezone to display events, supported timezones can be found at https://docs.rs/chrono-tz/0.5.1/chrono_tz/#modules
 event_timezone = "America/New_York"
 # Path to place logs in
diff --git a/src/discord/mod.rs b/src/discord/mod.rs
index 53a99d5..c7879a6 100644
--- a/src/discord/mod.rs
+++ b/src/discord/mod.rs
@@ -98,12 +98,9 @@ pub fn send_event_msg(
 
     let native_time = utc_time.with_timezone(&config.event_timezone);
 
-    let ping_roles = &config.ping_roles;
-    let mut ping_roles_msg = String::new();
-
-    for role in ping_roles {
-        ping_roles_msg.push_str(format!(" <@&{}>", role).as_str());
-    }
+    let ping_roles = config.ping_roles.clone().into_iter().map(|role| {
+        format!("<@&{}>", role)
+    }).collect::<Vec<String>>().join(" ");
 
     // Send message
     let msg = channel.id().send_message(&http, |m| {
@@ -122,7 +119,7 @@ pub fn send_event_msg(
                 .timestamp(utc_time.to_rfc3339())
                 .field("Location", &event.event_loc, true)
                 .field("Organizer", &event.organizer, true)
-        }).content(ping_roles_msg)
+        }).content(ping_roles)
     })?;
 
     if react {