chore: comments

src/App/App.jsx

@@ -70,7 +70,8 @@ export default function App() {
 
         setMethod("desktop");
       } else {
-        // If MTURK
+        // If using Mechanical Turk and PsiTurk
+        // TODO: Change config variable to psiturk? That's what user's are really using
         if (config.USE_MTURK) {
           /* eslint-disable */
           window.lodash = _.noConflict();
@@ -79,6 +80,7 @@ export default function App() {
           handleLogin("mturk", turkUniqueId);
           /* eslint-enable */
         } else if (config.USE_PROLIFIC) {
+          // TODO: Remove prolific check - we should always check process.env AND UrlSearchParams
           const pID = getProlificId();
           if (config.USE_FIREBASE && pID) {
             setMethod("firebase");
@@ -88,6 +90,7 @@ export default function App() {
             setIsError(true);
           }
         } else if (config.USE_FIREBASE) {
+          // TODO: There's a difference between PROLIFIC_ID in URL and PID/SID
           // Fill in login fields based on query parameters (may still be blank)
           const query = new URLSearchParams(window.location.search);
           const studyId = query.get("studyID");
@@ -102,7 +105,6 @@ export default function App() {
       }
     }
     setUpHoneycomb();
-    // eslint-disable-next-line
   }, []);
 
   /** VALIDATION FUNCTIONS */

src/trials/holdUpMarker.js

@@ -16,7 +16,7 @@ function holdUpMarker() {
     prompt: [p(language.trials.holdUpMarker)],
     choices: [language.prompts.continue.button],
     on_start: async () => {
-      // Ensure EEG is connected if using it
+      // Ensure event marker is connected if using it
       if (config.USE_EEG) await window.electronAPI.checkSerialPort();
     },
   };