From 3679a430f0a0ab33815e82cb2bdd1324a13a371d Mon Sep 17 00:00:00 2001 From: Gui Talarico Date: Wed, 12 Jun 2019 23:10:03 -0700 Subject: [PATCH 1/2] Clarify error message for missing api key If you accidentally make a request with a missing api key, error message is ambiguous. ```javascript // where process.env.AIRTABLE_KEY = null Airtable.configure({ endpointUrl: 'https://api.airtable.com', apiKey: process.env.AIRTABLE_KEY }) ... ``` Results in the error: ``` Error: API is required to connect to Airtable ``` In my case, I had forgotten to set the env var, so this throw me on a loop to figure out what `API is required` meant. It would be more helpful if it instead described: ``` API Key is required to connect to Airtable ``` }) --- lib/airtable.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/airtable.js b/lib/airtable.js index acf3583f..1d78b577 100644 --- a/lib/airtable.js +++ b/lib/airtable.js @@ -38,7 +38,7 @@ var Airtable = Class.extend({ this.requestTimeout = opts.requestTimeout || defaultConfig.requestTimeout; if (!this._apiKey) { - throw new Error('API is required to connect to Airtable'); + throw new Error('API Key is required to connect to Airtable'); } }, From bd1dbfd6c9de5c02c7d84de2da83d3d35538273e Mon Sep 17 00:00:00 2001 From: Gui Talarico Date: Thu, 13 Jun 2019 21:53:43 -0700 Subject: [PATCH 2/2] Revise error msg `An API key is required to connect to Airtable` --- lib/airtable.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/airtable.js b/lib/airtable.js index 1d78b577..57286653 100644 --- a/lib/airtable.js +++ b/lib/airtable.js @@ -38,7 +38,7 @@ var Airtable = Class.extend({ this.requestTimeout = opts.requestTimeout || defaultConfig.requestTimeout; if (!this._apiKey) { - throw new Error('API Key is required to connect to Airtable'); + throw new Error('An API key is required to connect to Airtable'); } },