Small fixes

+ Creating "ping_roles" more idiomatically
+ Updated readme

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

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 {