Drop http-status usage, Auth docs update (#84)

README.md

@@ -39,7 +39,7 @@ You just need node.js — and npm, obviously.
 
 ### Dependencies
 
-The dependency list is tiny — **node-fetch**, **http-status**, and **source-map-support**. They will be automatically installed when you install `wstrade-api` through `npm`.
+The dependency list is tiny — **node-fetch** and **source-map-support**. They will be automatically installed when you install `wstrade-api` through `npm`.
 
 ### Installing
 

docs/auth/README.md

@@ -78,7 +78,7 @@ auth.login(email, password) -> Promise<void>
 Attempts to login to the Wealthsimple Trade platform using the email and password combination. 
 If the login was successful, `auth.tokens` will be populated with the retrieved OAuth2.0 tokens and access token expiry time. Otherwise, the return promise is rejected with the appropriate error.
 
-An OTP provider must be configured beforehand with the auth.`on` API for the `otp` event. If the `otp` event was registered with a string literal, it is assumed that you have already obtained the OTP manually and it will be passed along as-is. Otherwise, if you specified a function handler, it is assumed that your handler will automatically retrieve the OTP after we attempt a login without an OTP.  See examples for practical explanation.
+An OTP provider must be configured beforehand with the auth.`on` API for the `otp` event. If the `otp` event was registered with a string literal, it is assumed that you have already obtained the OTP somehow and it will be passed along as-is. Otherwise, if you specified a function handler, it is assumed that your handler will automatically retrieve the OTP after we attempt a login without an OTP. Your OTP function should return the OTP code as a string literal. See examples for practical explanation.
 
 **Failure reasons**
 * Incorrect email/password combination

index.d.ts

@@ -8,7 +8,7 @@ declare namespace Trade {
   };
 
   // Supported exchanges
-  type Exchange = 'NYSE' | 'NASDAQ' | 'TSX' | 'TSX-V' | 'NEO';
+  type Exchange = 'NYSE' | 'NASDAQ' | 'TSX' | 'TSX-V' | 'NEO' | 'CC';
 
   /**
    * A ticker may be provided to the API as a string or an object with

package.json

@@ -1,8 +1,9 @@
 {
   "name": "wstrade-api",
-  "version": "1.3.0",
+  "version": "1.3.1",
   "description": "Wealthsimple Trade API Wrapper",
   "main": "dist.js",
+  "types": "index.d.ts",
   "scripts": {
     "bundle": "webpack",
     "lint": "npx eslint --fix src/ --ignore-pattern __test__",
@@ -42,7 +43,6 @@
     "webpack-cli": "^4.2.0"
   },
   "dependencies": {
-    "http-status": "^1.4.2",
     "node-fetch": "^2.6.1",
     "source-map-support": "^0.5.19"
   }

src/network/https.js

@@ -1,10 +1,11 @@
 import fetch, { Headers } from 'node-fetch';
-import status from 'http-status';
 import customHeaders from '../headers';
 import auth from '../auth';
 import { configEnabled } from '../config';
 import implicitTokenRefresh from '../optional/implicit-token-refresh';
 
+const [HTTP_OK, HTTP_CREATED] = [200, 201];
+
 /*
  * Complete the URL by filling the parameter placeholders with the
  * data arguments.
@@ -80,7 +81,7 @@ export default async function handleRequest(endpoint, data) {
   // Submit the HTTP request to the Wealthsimple Trade Servers
   const response = await talk(endpoint, data);
 
-  if ([status.OK, status.CREATED].includes(response.status)) {
+  if ([HTTP_OK, HTTP_CREATED].includes(response.status)) {
     return endpoint.onSuccess(response);
   }
 

webpack.config.js

@@ -19,12 +19,6 @@ module.exports = {
             amd: 'node-fetch',
             root: '_',
         },
-        "http-status": {
-            commonjs: 'http-status',
-            commonjs2: 'http-status',
-            amd: 'http-status',
-            root: '_',
-        },
         "source-map-support": {
             commonjs: 'source-map-support',
             commonjs2: 'source-map-support',