Should I be writing JS like this?

example_pattern.js

// A variable to know when to exit the 'inner' function. When entered = data.length, we know we can safely continue.
var entered = 0
var mah_list = []

async function inner(d, dlength){
  // SQL2
  setTimeout(function(){
    console.log('inner', d, dlength)
    mah_list.push(d)
    
    entered++;
    if (entered==dlength) {
      console.log("All done");
      final(mah_list);
    }
  }, 500, d)
}

async function outer(){
  // SQL1
  var data = [1, 2, 3, 4]

  for(d in data) {
    console.log('outer', d, entered)
    inner(d, data.length)
  }
}

outer()

console.log('here already?')

final = function (some_list) {
  console.log('continuing')
  console.log(some_list)
}