Skip to main content

Script not running, with no obvious errors?

R
Forum|alt.badge.img+6

Hi everyone,

I’m performing a vlookup essentially at a spesific time each day. It worked the first couple of times, but now keeps giving me an error, any advice/guidance would be greatly appreciated!



    //"Time Sheet" contains values on which you want to run the vlookup
    let mainTable = base.getTable("Time Sheet");
    let mainTableRecords = await mainTable.selectRecordsAsync({fields:["Title Name"]});

    //"Volunteers" contains range to search in
    let lookupTable = base.getTable("Volunteers");
    let lookupRangeRecords = await lookupTable.selectRecordsAsync({fields:["Name"]});

    //"Title Name" has the values you want to look up 
    for (let record of mainTableRecords.records) {  
        let lookupValue = record.getCellValue("Title Name");

        //"Name" is the range to search in
        for (let rangeRecord of lookupRangeRecords.records) {
            if (rangeRecord.getCellValue("Name") === lookupValue) {
                let linkID = rangeRecord.id;
        
                //"Volunteers" is column name from mainTable which should contain the link
                await mainTable.updateRecordAsync(record, {
                    "Volunteers": [{id: linkID}]
                });
        }
        }
    }

2 replies

B
Forum|alt.badge.img+19
  • Bill_French
  • Inspiring
  • 3264 replies
  • February 6, 2022

We can’t run your code so you have to be more transparent about the error and perhaps the entire code.

Your message has conflicting data -

  • no obvious errors
  • gives me an error

Which is it?

Recommendations:

  1. Add try-catch to understand where the errors are occurring and why
  2. Add console.log() points to capture the state of the various loops.
  3. Avoid nested loop lookups (this might help)
kuovonne
Forum|alt.badge.img+27
  • kuovonne
  • Brainy
  • 6001 replies
  • February 6, 2022
Bill_French wrote:

We can’t run your code so you have to be more transparent about the error and perhaps the entire code.

Your message has conflicting data -

  • no obvious errors
  • gives me an error

Which is it?

Recommendations:

  1. Add try-catch to understand where the errors are occurring and why
  2. Add console.log() points to capture the state of the various loops.
  3. Avoid nested loop lookups (this might help)

These are great troubleshooting suggestions. For people new to coding, I suggest adding console.log as a first step before using try-catch. While try-catch is extremely useful when dealing with an external system that can produce temperamental results, it isn’t as helpful when trying to debug a system without outside dependencies.

It would be easier to troubleshoot with more information, such as the actual behavior being seen.

Is this a automation script?
When the script stopped working, were there more records to be updated compared to when the script worked?
Are you finding that some records are updated, but not all of them?
Is the base schema (field types) exactly the same as it was when the script worked?

If the answers to all of these questions is yes, it is likely that you ran into a time limit for automation scripts. The solution is to rewrite the script so that it does not have nested loops and to update records in batches. Take a look at Bill’s link.

Reply


Most helpful members this week

Terms & Conditions