Bug 579688: Notification about study about to start should say "about to start", not "is starting". r=dtownsend

browser/app/profile/extensions/testpilot@labs.mozilla.com/content/experiment-page.js

@@ -367,6 +367,8 @@ var stringBundle;
     var contentDiv = document.getElementById("experiment-specific-text");
     var dataPrivacyDiv = document.getElementById("data-privacy-text");
     // Get experimentID from the GET args of page
+    // TODO no reason actually to do parseInt here -- all it accomplishes
+    // is preventing us from using non-numeric study IDs.
     var eid = parseInt(getUrlParam("eid"));
     var experiment = TestPilotSetup.getTaskById(eid);
     if (!experiment) {

browser/app/profile/extensions/testpilot@labs.mozilla.com/modules/setup.js

@@ -561,7 +561,7 @@ let TestPilotSetup = {
 	    this._showNotification(
 	      task, false,
 	      this._stringBundle.formatStringFromName(
-		"testpilot.notification.newTestPilotStudy.message",
+		"testpilot.notification.newTestPilotStudy.pre.message",
 		[task.title], 1),
 	      this._stringBundle.GetStringFromName(
 		"testpilot.notification.newTestPilotStudy"),

browser/locales/en-US/feedback/main.properties

@@ -83,7 +83,7 @@ testpilot.notification.thankYouForUploadingData.message = Thank you for uploadin
 testpilot.notification.readyToSubmit = Ready to Submit
 testpilot.notification.readyToSubmit.message = The Test Pilot "%S" study is finished gathering data and is ready to submit.
 testpilot.notification.newTestPilotStudy = New Test Pilot Study
-testpilot.notification.newTestPilotStudy.message = The Test Pilot "%S" study is now beginning.
+testpilot.notification.newTestPilotStudy.pre.message = The Test Pilot "%S" study is about to begin.
 testpilot.notification.newTestPilotSurvey = New Test Pilot Survey
 testpilot.notification.newTestPilotSurvey.message = The Test Pilot "%S" survey is available.
 testpilot.notification.newTestPilotResults = New Test Pilot Results