diff --git a/packages/server/src/integrations/googlesheets.ts b/packages/server/src/integrations/googlesheets.ts index 5416f07987..57b6682cc8 100644 --- a/packages/server/src/integrations/googlesheets.ts +++ b/packages/server/src/integrations/googlesheets.ts @@ -287,7 +287,6 @@ class GoogleSheetsIntegration implements DatasourcePlus { ): Promise { // not fully configured yet if (!this.config.auth) { - // TODO(samwho): is this the correct behaviour? return { tables: {}, errors: {} } } await this.connect() diff --git a/packages/server/src/integrations/utils.ts b/packages/server/src/integrations/utils.ts index 0ce067e967..79b18e767c 100644 --- a/packages/server/src/integrations/utils.ts +++ b/packages/server/src/integrations/utils.ts @@ -305,7 +305,7 @@ export function finaliseExternalTables( for (let [name, table] of Object.entries(tables)) { finalTables[name] = copyExistingPropsOver(name, table, entities, tableIds) } - // sort the tables by name (TODO(samwho): why?) + // sort the tables by name, this is for the UI to display them in alphabetical order return Object.entries(finalTables) .sort(([a], [b]) => a.localeCompare(b)) .reduce((r, [k, v]) => ({ ...r, [k]: v }), {})