GASによるチャットワークへの通知

Question

度々失礼致します、実現させたいことは
①getMailのトリガーで1分ごとにGmailの内容を読み取り、スプレッドシートに記入する
②そのスプレッドシートに記入した内容をchatNotifyのトリガーで「スプレッドシートが変更された時に」通知の設定を行うことです。

現在、以下のコードで①までは問題なく実現できるのですが、スプレッドシートに書き出された物をチャットワークに通知がされません。
ご教授お願い致します。どうぞよろしくお願い致します。

function chatNotify() {
  var client = ChatWorkClient.factory({token: "77777777777777777777"});
  var objSpreadsheet = SpreadsheetApp.getActiveSpreadsheet();
  var objSheet = objSpreadsheet.getSheetByName("シート1");
  var number1 = objSheet.getRange('A1').getValue();
  client.sendMessage({
    room_id: 1111111111, 
    body:number1　+ "に注文がありました"
   
  });   
}



function getMail(){
  var objSpreadsheet = SpreadsheetApp.getActiveSpreadsheet();
  var objSheet = objSpreadsheet.getSheetByName("シート1");
  var sheet = SpreadsheetApp.setActiveSheet(objSheet);


   var start = 0;
  var max = 500;
  var threads = GmailApp.search('label:重要 is:unread',start,max);
  var row = sheet.getLastRow() + 1;

 for(var n in threads){
    var thd = threads[n];
    var msgs = thd.getMessages();

    for(var　m in msgs){
      var msg = msgs[m];
      var body = msg.getBody();
      var date = msg.getDate();
      sheet.appendRow([date]);
}
  thd.markRead();  
  Utilities.sleep(1000);
  }
}

Answer 1

getMail内でchatNotifyを呼び出せばいいだけなのでは？
わざわざ2つにわけているのは何か意図があってのことなのでしょうか？

function chatNotify() {
  var client = ChatWorkClient.factory({token: "77777777777777777777"});
  var objSpreadsheet = SpreadsheetApp.getActiveSpreadsheet();
  var objSheet = objSpreadsheet.getSheetByName("シート1");
  var number1 = objSheet.getRange('A1').getValue();
  client.sendMessage({
    room_id: 1111111111, 
    body:number1　+ "に注文がありました"
  });   
}



function getMail(){
  var objSpreadsheet = SpreadsheetApp.getActiveSpreadsheet();
  var objSheet = objSpreadsheet.getSheetByName("シート1");
  var sheet = SpreadsheetApp.setActiveSheet(objSheet);


  var start = 0;
  var max = 500;
  var threads = GmailApp.search('label:重要 is:unread',start,max);
  var row = sheet.getLastRow() + 1;

  for(var n in threads){
    var thd = threads[n];
    var msgs = thd.getMessages();

    for(var　m in msgs){
      var msg = msgs[m];
      var body = msg.getBody();
      var date = msg.getDate();
      sheet.appendRow([date]);
      chatNotify(); // ここで呼び出し
    }
    thd.markRead();  
    Utilities.sleep(1000);
  }
}

Comments

[mogumog]

ありがとうございます！早速試してみようと思います！

Answer 2

うろ覚えなのですが回答ついてないので。
スクリプトの処理からトリガーが発動しなかったように思います。

今回のものだと時間がかかりそうなので、通知の方を時間主導トリガーにしてはどうでしょうか？

Comments

[mogumog]

ありがとうございます、一度時間主導型で試してみようと思います。

[papinianus]

この回答はスリープしているコードで複雑なことをしたくない観点を前提にしています。
よく分からないのですがなぜA1固定なのでしょう。一度しか走らせなくて良いなら最後の閉じカッコ}の直前でchatNotifyを実行すべきですし、メールに応じてやるならA1固定は適切ではない。