Merge pull request #98 from mxmeinhold/develop

Implements sending slack message upon 💯

Author: JoelEager

config.env.py

@@ -21,3 +21,6 @@
 # LDAP config
 LDAP_BIND_DN = environ.get("PACKET_LDAP_BIND_DN", None)
 LDAP_BIND_PASS = environ.get("PACKET_LDAP_BIND_PASS", None)
+
+# Slack URL for pushing to #general
+SLACK_WEBHOOK_URL = environ.get("PACKET_SLACK_URL", None)

packet/models.py

@@ -103,6 +103,14 @@ def did_sign(self, username, is_csh):
         # The user must be a misc CSHer that hasn't signed this packet or an off-floor freshmen
         return False
 
+
+    def is_100(self):
+        """
+        Checks if this packet has reached 100%
+        """
+        return self.signatures_required().total == self.signatures_recieved().total
+
+
     @classmethod
     def open_packets(cls):
         """

packet/routes/api.py

@@ -1,5 +1,5 @@
 from packet import app, db
-from packet.utils import before_request, packet_auth
+from packet.utils import before_request, packet_auth, notify_slack
 from packet.models import Packet, MiscSignature
 
 
@@ -10,22 +10,29 @@ def sign(packet_id, info):
     packet = Packet.by_id(packet_id)
 
     if packet is not None and packet.is_open():
+        was_100 = packet.is_100()
         if app.config["REALM"] == "csh":
             # Check if the CSHer is an upperclassman and if so, sign that row
             for sig in filter(lambda sig: sig.member == info["uid"], packet.upper_signatures):
                 sig.signed = True
                 db.session.commit()
+                if not was_100 and packet.is_100():
+                    notify_slack(packet.freshman.name)
                 return "Success: Signed Packet: " + packet.freshman_username
 
             # The CSHer is a misc so add a new row
             db.session.add(MiscSignature(packet=packet, member=info["uid"]))
             db.session.commit()
+            if not was_100 and packet.is_100():
+                notify_slack(packet.freshman.name)
             return "Success: Signed Packet: " + packet.freshman_username
         else:
             # Check if the freshman is onfloor and if so, sign that row
             for sig in filter(lambda sig: sig.freshman_username == info["uid"], packet.fresh_signatures):
                 sig.signed = True
                 db.session.commit()
+                if not was_100 and packet.is_100():
+                    notify_slack(packet.freshman.name)
                 return "Success: Signed Packet: " + packet.freshman_username
 
     return "Error: Signature not valid.  Reason: Unknown"

packet/utils.py

@@ -85,3 +85,13 @@ def wrapped_function(*args, **kwargs):
         return func(*args, **kwargs)
 
     return wrapped_function
+
+def notify_slack(name: str):
+    """
+    Sends a congratulate on sight decree to Slack.
+    """
+    if app.config["SLACK_WEBHOOK_URL"] is None:
+        print("SLACK_WEBHOOK_URL not configured, not sending message to slack.")
+        return
+    msg = f'{name} got :100: on packet. Shower on sight.'
+    requests.put(app.config["SLACK_WEBHOOK_URL"], json={'text':msg})