GASでADパスワードの有効期限が近付いてきた従業員にslackのDMで通知をしたい。

Question

先日同様のタイトルで質問させていただいたのですがパスワードの有効期限が14日以下のユーザーの抽出まではうまくいったのですが、slackのDMで通知をする際、14日以下のユーザー情報がすべてDMの内容に記載されてしまいます。
これを14日以下になっている該当のユーザーに自分の情報のみを通知させたいと考えています。

プログラム自体素人の為、まったくいい方法が分からず困っています。

現在は以下のように作成しています。

function alertContract() {
    var mySheet = SpreadsheetApp.openById('○○○○').getSheetByName('××××'); //スプレッドシートを取得
    var lastRow = mySheet.getLastRow();　//スプレッドシートの最終行を取得
    var today = new Date(); //今日の日付を取得

  
    /* 契約見直日まで14日切っていたらSlack投稿 */
    for (var i = 2; i <= lastRow; i++) {
        var Di = mySheet.getRange(i, 1).getValue(); //ユーザー名を取得
        var days = mySheet.getRange(i, 4).getValue();　//期限切れ日数を取得
        var dateHi = new Date(mySheet.getRange(i, 2).getValue()); //有効期限を取得
        if (days <= 14) { //2週間前に告知
        
        var strText = "ADアカウントのパスワード期限が迫っています。パスワードの変更をお願いします。" + Di + " " + "有効期限切れまで" + days + "日前"　+ "\nhttps://docs.google.com/document/d/1P7L_"

          postDM(strText);
        }
    }
}


const slack_app_token = "xoxb-△△△△";
 
//botからDMを送る
function postDM(strText){
    var mySheet = SpreadsheetApp.openById('○○○○').getSheetByName('××××'); //スプレッドシートを取得
    var lastRow = mySheet.getLastRow();　//スプレッドシートの最終行を取得
 
  var message = strText   
  //【処理1】DMを開き、チャンネルIDを取得する
  const member_id = mySheet.getRange(28,5 ).getValue(); //メンバーIDを指定
  const channel_id = getChannelID_(member_id);
  
  //【処理2】指定の[チャンネルID]にDMを送信する
  const message_options = {
    "method" : "post",
    "contentType": "application/x-www-form-urlencoded",
    "payload" : {
      "token": slack_app_token,
      "channel": channel_id,
      "text": message
    }
  };
  
  //必要scope = chat:write
  const message_url = 'https://slack.com/api/chat.postMessage';
  UrlFetchApp.fetch(message_url, message_options);
  
}
 
 
/**
* メンバーIDを受け取りチャンネルIDを返す
* 
* @param {string} メンバーID 
* @return {string} チャンネルID
*/
function getChannelID_(member_id) {
 
  const options = {
    "method" : "post",
    "contentType": "application/x-www-form-urlencoded",
    "payload" : {
      "token": slack_app_token,
      "users": member_id
    }
  }
  
  //必要scope = im:write
  const url = 'https://slack.com/api/conversations.open';
  const response = UrlFetchApp.fetch(url, options);
  
  const obj = JSON.parse(response);
  console.log(obj);
  
  return obj.channel.id;
 
}

考えとしては条件に該当するユーザーのslackユーザーIDを取得してDMを送るようにすればいいのではないかと考えています。

どなたかご教授いただけませんでしょうか。
宜しくお願い致します。

Answer 1

postDM()にユーザーIDを渡すようにしたらだうまくいかないでしょうか。

function alertContract() {
(略)


    /* 契約見直日まで14日切っていたらSlack投稿 */
    for (var i = 2; i <= lastRow; i++) {
        var Di = mySheet.getRange(i, 1).getValue(); //ユーザー名を取得
        var days = mySheet.getRange(i, 4).getValue();　//期限切れ日数を取得
        var dateHi = new Date(mySheet.getRange(i, 2).getValue()); //有効期限を取得
+       var user_id = mySheet.getRange(i, 3).getValue(); //ユーザーIDを取得
        if (days <= 14) { //2週間前に告知

          var strText = "ADアカウントのパスワード期限が迫っています。パスワードの変更をお願いします。" + Di + " " + "有効期限切れまで" + days + "日前"　+ "\nhttps://docs.google.com/document/d/1P7L_evWv8tjBG854tBiu6dGFEW1H3xsbaNRIKlY2dWo/edit?usp=sharing"

-         postDM(strText);
+         postDM(strText, user_id);
        }
    }
}

const slack_app_token = "xoxb-△△△△";

//botからDMを送る
-function postDM(strText){
+function postDM(strText, user_id){
-   var mySheet = SpreadsheetApp.openById('○○○○').getSheetByName('××××'); //スプレッドシートを取得
-   var lastRow = mySheet.getLastRow();　//スプレッドシートの最終行を取得

  var message = strText   
  //【処理1】DMを開き、チャンネルIDを取得する
- const member_id = mySheet.getRange(28,5 ).getValue(); //メンバーIDを指定
- const channel_id = getChannelID_(member_id);
+ const channel_id = getChannelID_(user_id);

(以下略)

Comments

[kaguramiyabi]

qnoir さん

ご回答いただきありがとうございます。

ご教授いただきました内容に修正いたしましたところ「TypeError: Cannot read property 'id' of undefined」となってしまい動作しない状況です。
エラーとなっているコードはreturn obj.channel.id;となります。

お手数ですが引き続き宜しくお願い致します。

[kaguramiyabi]

qnoir さん回答を元とに下記コードを修正しましたがエラーは改善されませんでした。
変更前
function getChannelID_(member_id) {

const options = {
"method" : "post",
"contentType": "application/x-www-form-urlencoded",
"payload" : {
"token": slack_app_token,
"users": member_id
}
}

//必要scope = im:write
const url = 'https://slack.com/api/conversations.open';
const response = UrlFetchApp.fetch(url, options);

const obj = JSON.parse(response);
console.log(obj);

return obj.channel.id;

変更後
function getChannelID_(channel_id) {

const options = {
"method" : "post",
"contentType": "application/x-www-form-urlencoded",
"payload" : {
"token": slack_app_token,
"users": channel_id
}
}

//必要scope = im:write
const url = 'https://slack.com/api/conversations.open';
const response = UrlFetchApp.fetch(url, options);

const obj = JSON.parse(response);
console.log(obj);

return obj.channel.id;

[kaguramiyabi]

試しにuser_idを自身のユーザーIDで直接指定して行ったところエラーは出ずにDMが届きましたがやはり有効期限が14日以下のユーザー情報がすべて掲載されてしまいます。

[退会済みユーザー]

質問文に記載のの修正前のコードで
postDM()の最初の2行
　var mySheet = SpreadsheetApp.openById('○○○○').getSheetByName('××××'); //スプレッドシートを取得
var lastRow = mySheet.getLastRow();　//スプレッドシートの最終行を取得

ここで「DM送付先のユーザーIDを取得している」と推察したのですが、違いますでしょうか？

また、前回の質問に記載されていた画像から、スプレッドシートの3列目に各々のユーザーIDが記載されていると推察したのですが、違うのでしょうか？

考え方として、ユーザーごとにDM送付先のユーザーIDを切り替えればいいだけだと思いました。
修正前の質問のコードでは、１つのユーザーIDに対してすべてのデータを送付しているので「有効期限が14日以下のユーザー情報がすべて掲載されてしまう」という事態が発生しています。

修正後のコードは、ユーザーごとにユーザーIDを切り替えるつもりで書きました。
動作が正しくても。そもそもユーザーID（スプレッドシートの3列目に記載されているユーザーID）が誤っているか、無効であれば、エラーが発生します。

[kaguramiyabi]

qnoirさん
ご回答ありがとうございます。
var mySheet = SpreadsheetApp.openById('○○○○').getSheetByName('××××'); //スプレッドシートを取得
var lastRow = mySheet.getLastRow();　//スプレッドシートの最終行を取得
上記のコードではスプレッドシート内にユーザーIDを3行目に記載しているのでユーザーIDを取得させる為に記載していました。
3行目のユーザーIDは関数で該当ユーザーのユーザーIDを取得して表示させています。ただ、実際の3行目を指定してしまうと実際にDMで通知が行ってしまうので別途5行目に自身のユーザーIDのみ記載し、自身のADアカウントの有効期限を2週間以下に設定してご教授いただいたコードに修正してテストを行いましたが、エラーとなってしまい通知されませんでした。

『考え方として、ユーザーごとにDM送付先のユーザーIDを切り替えればいいだけだと思いました。
修正前の質問のコードでは、１つのユーザーIDに対してすべてのデータを送付しているので「有効期限が14日以下のユーザー情報がすべて掲載されてしまう」という事態が発生しています。』
とのことですが、現在はテストとして自身のみに通知を行っているのですがこれを対象者へ通知した場合は自分が想定する対象ユーザーのみのデータが表示されるのでしょうか。

[kaguramiyabi]

本件について複数のユーザーにDMで通知する際は該当のユーザーのみの情報が表示されることを確認しました。確認方法としては対象となるデータをデータ列の最初に移動させコードを動かしたところDMが送られ自分のデータが記載されていました。
しかし、AD上にユーザーは存在していますが、使用していないユーザーもあるので使用していないユーザーのslackユーザーIDは空白となっています。昨日記載したエラーは空白となっていることでエラーになってしまっているのではないかと考えています。
ただ、5行目の空白内にａを打ってvar user_id = mySheet.getRange(i, 5).getValue();で指定し、実行しても自分のデータのところまでは届かずエラーとなってしまいます。

[kaguramiyabi]

本件ですが、エラーの原因が判明いたしました。qnoirさんのおっしゃる通り『そもそもユーザーID（スプレッドシートの3列目に記載されているユーザーID）が誤っているか、無効であれば、エラーが発生します。』が原因でユーザーIDが空白または正しくない場合はエラーとなって動作が止まってしまうことを確認しました。その為、運用するときは3行目にはすべて正しいユーザーIDを取得して使用していないADアカウントに関してはif (days <= 14 && days >=-30)として14日以下・-30日以上のユーザーのみが対象となるように修正いたしました。ご協力いただきましてありがとうございました。

[退会済みユーザー]

時間がなく、すぐ対応できず申し訳ありませんでした。解決されたのであればよかったです。