feat: rename rename OAuthOnAuthorizeToken to OAuthTokenInput; fix flo…

…w warnings
goabstract · Jul 15, 2020 · efcfe27 · efcfe27
1 parent a98f43a
commit efcfe27
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 8 deletions.
diff --git a/src/endpoints/OAuth.js b/src/endpoints/OAuth.js
@@ -2,7 +2,7 @@
 import { BaseError } from "../errors";
 import type {
   OAuthAuthorizeInput,
-  OAuthOnAuthorizeToken,
+  OAuthTokenInput,
   TokenResponseData
 } from "../types";
 import Endpoint from "../endpoints/Endpoint";
@@ -11,19 +11,20 @@ import Client from "../Client";
 export default class OAuth extends Endpoint {
   name = "oauth";
 
-  getToken(input: OAuthOnAuthorizeToken) {
+  getToken(input: OAuthTokenInput) {
     const clientId = input.clientId || this.options.clientId;
     const clientSecret = input.clientSecret || this.options.clientSecret;
     let redirectUri = input.redirectUri || this.options.redirectUri;
     const { authorizationCode } = input;
 
     const body = new URLSearchParams();
 
-    // $FlowFixMe
+    if (!clientId || !clientSecret || !redirectUri) {
+      throw new Error("OAuthTokenInput required");
+    }
+
     body.append("client_id", clientId);
-    // $FlowFixMe
     body.append("client_secret", clientSecret);
-    // $FlowFixMe
     body.append("redirect_uri", redirectUri);
 
     body.append("code", authorizationCode);

diff --git a/src/types.js b/src/types.js
@@ -160,7 +160,7 @@ export type CommandOptions = {
   analyticsCallback: AnalyticsCallback,
   apiUrl: string | Promise<string>,
   clientId?: string,
-  clientSecret?: string | Promise<string>,
+  clientSecret?: string,
   objectUrl: string | Promise<string>,
   previewUrl: string | Promise<string>,
   shareId?: () => Promise<string | ShareDescriptor | ShareUrlDescriptor | void>,
@@ -462,7 +462,6 @@ export type Membership = {
 };
 
 export type Organization = {
-  features: { [feature: string]: boolean },
   hasBillingInfo?: boolean,
   id: string,
   isUsernameOrganization?: boolean,
@@ -1682,7 +1681,7 @@ export type OAuthAuthorizeInput = {
   state: string
 };
 
-export type OAuthOnAuthorizeToken = {
+export type OAuthTokenInput = {
   redirectUri: string,
   clientSecret: string,
   clientId: string,