solving mail threads issue #1

code.gs

@@ -1,51 +1,57 @@
+// Filter all travel invoice from GOJEK, for example:
+// Matches: from:(no-reply@invoicing.go-jek.com)
+// Do this: Skip Inbox, Apply label "yourlabel"
+
 // Change two necessary variables below:
-// 1. label for email will be pulled 
-var mailLabel = "Invoice/Gojek/Travel"
+// 1. yourlabel for email will be pulled 
+var mailLabel = "Invoice/Gojek/Travel";
 // 2. allowed email for security reason
-var adminMail = "herlandi.27@gmail.com"
+var adminMail = "herlandi.27@gmail.com";
 
 // Add Spreadsheet's UI menu
 function onOpen(e) {
   var ui = SpreadsheetApp.getUi();
-  ui.createMenu("Pull mails").addItem("Get travel details", "getTravel").addToUi();
+  ui.createMenu("Pull mails").addItem("Get travel details", "getMsg").addToUi();
 }
 
+/*
+function test(){
+  const travelThread = GmailApp.getUserLabelByName(mailLabel).getThreads();
+  Logger.log(travelMessage = travelThread[0].getMessages()[1].getPlainBody());
+
+} */
+
 // Get mails
-function getTravel() {
+function getMsg() {
 
   // security
   if (Session.getActiveUser().getEmail() != adminMail) {
-    Browser.msgBox("Please LogIn as a valid users!");
+    Browser.msgBox("Please Login as a valid users!");
     return;
   }
 
-  // define label and its thread
-  var travelLabel = GmailApp.getUserLabelByName(mailLabel);
-  var travelThread = travelLabel.getThreads();
-
-  for(var i = travelThread.length-1; i >= 0; i--) {
+  // index mails suitable to given label and their threads
+  var travelThread = GmailApp.getUserLabelByName(mailLabel).getThreads();
+
+  for(var i = 0; i < travelThread.length; i++) {
 
     var travelMessage = travelThread[i].getMessages();
 
     for(var j = 0; j < travelMessage.length; j++) {
-      var travelMessage = travelMessage[j];
-      extractDetails(travelMessage);
-    }
-  }
 
-}
+      var travelDetail = travelMessage[j];
 
-// Write on Spreadsheets
-function extractDetails(travelMessage) {
-
-  // define fileds column
-  var datetime = travelMessage.getDate();
-  var subject = travelMessage.getSubject();
-  var sender = travelMessage.getFrom();
-  var content = travelMessage.getPlainBody();
+      // define fileds column
+      var datetime = travelDetail.getDate();
+      var subject = travelDetail.getSubject();
+      var sender = travelDetail.getFrom();
+      var content = travelDetail.getPlainBody();
 
-  var activeSheet = SpreadsheetApp.getActiveSpreadsheet().getActiveSheet();
-  activeSheet.appendRow([datetime, sender, subject, content]);
+      // append to new rows
+      var activeSheet = SpreadsheetApp.getActiveSpreadsheet().getActiveSheet();
+      activeSheet.appendRow([datetime, sender, subject, content]);
 
+    }
+  }
 }