Added logging for DB writes

Author: JoelEager

packet/routes/api.py

@@ -15,17 +15,21 @@ def sign(packet_id, info):
             for sig in filter(lambda sig: sig.member == info["uid"], packet.upper_signatures):
                 sig.signed = True
                 db.session.commit()
+                app.logger.info("Member {} signed packet {} as an upperclassman".format(info["uid"], packet_id))
                 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()
+            app.logger.info("Member {} signed packet {} as a misc".format(info["uid"], packet_id))
             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()
+                app.logger.info("Freshman {} signed packet {}".format(info["uid"], packet_id))
                 return "Success: Signed Packet: " + packet.freshman_username
 
+    app.logger.warn("Failed to add {}'s signature to packet {}".format(info["uid"], packet_id))
     return "Error: Signature not valid.  Reason: Unknown"

packet/routes/freshmen.py

@@ -45,6 +45,7 @@ def submit_essays(packet_id, info=None):
         packet.info_achieve = request.form.get("info_achieve", None)
 
         db.session.commit()
+        app.logger.info("Updated essays for {}".format(info["uid"]))
         return redirect(url_for("essays", packet_id=packet_id), 302)
     else:
         return redirect(url_for("index"), 302)