Closed JohnDamilola closed 1 year ago
@JohnDamilola Brad and I reviewed this PR. Collin and I had misunderstood what was going on. Please disregard above comment and accept my apologies. As high-level feedback on this PR:
loadConfig().then((config) => {main(config);})
leads to a race condition where we cannot receive events until we've loaded the config. That means we might drop events if we open up a new browser window pointed at Google Docs depending on order of events.
@JohnDamilola Would it be possible to please break this up into two pull requests, one with functionality changes, and one with formatting? It's very hard to see what changed if it's all in one commit.