diff --git a/wrenchboard/src/shared_tool/email.cc b/wrenchboard/src/shared_tool/email.cc
index 2969cf6e..a11661f0 100644
--- a/wrenchboard/src/shared_tool/email.cc
+++ b/wrenchboard/src/shared_tool/email.cc
@@ -83,7 +83,7 @@ long promo_email(long mailtype, CVars in, CVars &out){
 
         case WRB_CRONJOB_PROCESS_PROMO:
             vars2form(in, form);
-            form.LetStr("email", out["email"]);
+            form.LetStr("email", in["email"]);
             form.Email("PROMO/promo_alert.mailfile");
             break;
     }
diff --git a/wrenchboard/src/shared_tool/site_crons.cc b/wrenchboard/src/shared_tool/site_crons.cc
index 0db79a2f..f5189ae3 100644
--- a/wrenchboard/src/shared_tool/site_crons.cc
+++ b/wrenchboard/src/shared_tool/site_crons.cc
@@ -218,7 +218,10 @@ long  RefreshPromoActivities(CVars in, CVars &out){
                 resF = pgsql_query("SELECT status FROM members_pending where LOWER(username)= LOWER('%s')", rec["email"].c_str());
                 if (resF != NULL && pgsql_num_rows(resF) > 0) {
                    pgsql_query("UPDATE promo_member SET updated = now(), status=2 WHERE id = %lu ", rec["id"].Long()); // mo
-                   RegisterPromoAction(rec);
+                    CVars yp;
+                    yp["category"] ="SIGNUP";
+                    yp["description"] = "Sign up was started";
+                    RegisterPromoAction(yp);
                 }
             }
         }
@@ -278,7 +281,12 @@ long RegisterPromoPoint(CVars in){
 
 
 
-  RegisterPromoAction(in);
+
+
+    CVars yp;
+    yp["category"] ="SIGNUP_POINT";
+    yp["description"] = "Sign up was completed";
+    RegisterPromoAction(yp);
   return 0;
 }