Merge pull request #56 from deskbookers/develop

Payments SetDefaultSource bug

Author: rafaeldelboni

CHANGELOG.md

@@ -4,6 +4,10 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](http://keepachangelog.com/)
 and this project adheres to [Semantic Versioning](http://semver.org/).
 
+## 5.2.1 - 2017-08-21
+### Changes
+- Fixed `changeDefaultSource` with no body bug.
+
 ## 5.2.0 - 2017-08-07
 ### Changes
 - Added `payments` resource.

docs/payments.md

@@ -1,109 +1,5 @@
 # Payments
 
-## `invoices.list({limit, lastId})`
-List all avaiable invoices for a given `limit`, `lastId`
-
-#### Params
-
-| Name    | Type   | Required | Description               |
-| ------- | ------ | -------- | ------------------------- |
-| limit   | int    | No       | Limit results per request |
-| lastId  | string | No       | Used for Pagination       |
-
-```js
-const data = await deskbookers.payments.invoices.list({})
-```
-
-## `invoices.get({invoiceId})`
-List all avaiable invoices for a given `invoiceId`
-
-#### Params
-
-| Name      | Type   | Required | Description             |
-| --------- | ------ | -------- | ----------------------- |
-| invoiceId | int    | Yes      | Invoice Id              |
-
-```js
-const data = await deskbookers.payments.invoices.get(
-  {invoiceId: 'in_1AmlLiGAwNnYLy52pW4WN9p1'}
-)
-```
-
-## `plans.list({limit, lastId})`
-List all avaiable plans for a given `limit`, `lastId`
-
-#### Params
-
-| Name    | Type   | Required | Description               |
-| ------- | ------ | -------- | ------------------------- |
-| limit   | int    | No       | Limit results per request |
-| lastId  | string | No       | Used for Pagination       |
-
-```js
-const data = await deskbookers.payments.plans.list({})
-```
-
-## `plans.get({planId})`
-List all avaiable plans for a given `planId`
-
-#### Params
-
-| Name      | Type   | Required | Description             |
-| --------- | ------ | -------- | ----------------------- |
-| planId    | int    | Yes      | Plan Id                 |
-
-```js
-const data = await deskbookers.payments.plans.get(
-  {planId: 'bookingTool-9900-1x-month-0'}
-)
-```
-
-## `plans.subscriptions({planId})`
-List all avaiable subscriptions for a given `planId`
-
-#### Params
-
-| Name      | Type   | Required | Description             |
-| --------- | ------ | -------- | ----------------------- |
-| planId    | int    | Yes      | Plan Id                 |
-
-```js
-const data = await deskbookers.payments.plans.get(
-  {planId: 'bookingTool-9900-1x-month-0'}
-)
-```
-
-## `subscriptions.list({limit, lastId, status})`
-List all avaiable subscriptions for a given `limit`, `lastId` and `status`
-
-#### Params
-
-| Name    | Type   | Required | Description               |
-| ------- | ------ | -------- | ------------------------- |
-| limit   | int    | No       | Limit results per request |
-| lastId  | string | No       | Used for Pagination       |
-| status  | string | No       | Used for filtering        |
-
-```js
-const data = await deskbookers.payments.subscriptions.list(
-  {limit: 1, status: 'active'}
-)
-```
-
-## `subscriptions.get({subscriptionId})`
-Get subscription for a given `subscriptionId`
-
-#### Params
-
-| Name           | Type   | Required | Description         |
-| -------------- | ------ | -------- | ------------------- |
-| subscriptionId | int    | Yes      | Subscription Id     |
-
-```js
-const data = await deskbookers.payments.subscriptions.get(
-  {subscriptionId: 'sub_B93SRrYAV6KF7V'}
-)
-```
 
 ## `users.get({userId})`
 Get user for a given `userId`
@@ -211,68 +107,6 @@ const data = await deskbookers.payments.users.createSource({
 })
 ```
 
-## `users.createSubscription({userId, subscription})`
-Create subscriptions for a given `userId`
-
-#### Params
-
-| Name           | Type   | Required | Description         |
-| -------------- | ------ | -------- | ------------------- |
-| userId         | string | Yes      | User Id             |
-| subscription   | object | Yes      | Subscription Object |
-
-#### Subscription
-
-| Name          | Type   | Required | Description           |
-| ------------- | ------ | -------- | --------------------- |
-| currency      | string | Yes      | Subscription currency |
-| userUid       | string | Yes      | Origin of 'features'  |
-| interval      | object | Yes      | Subscription interval |
-| email         | string | Yes      | Subscription email    |
-| vat           | int    | Yes      | Subscription vat      |
-| plans         | object | Yes      | Subscription Plans    |
-
-#### Interval
-
-| Name        | Type   | Required | Description             |
-| ----------- | ------ | -------- | ----------------------- |
-| period      | string | Yes      | Interval period         |
-| count       | int    | Yes      | Interval count          |
-
-#### plans
-
-| Name        | Type   | Required | Description             |
-| ----------- | ------ | -------- | ----------------------- |
-| feature     | string | Yes      | Plans feature           |
-| name        | string | Yes      | Plans name              |
-| amount      | float  | Yes      | Plans amount            |
-
-
-```js
-const data = await deskbookers.payments.users.createSubscription({
-  userId: 'rb6',
-  subscription: {
-    currency: 'eur',
-    userUid: 'features',
-    interval: { period: 'month', count: 1 },
-    email: '[email protected]',
-    vat: 21,
-    plans: [
-      {
-        feature: 'basic',
-        name: 'Deskbookers for Partners Basic',
-        amount: 10.0
-      },
-      {
-        feature: 'bookingTool',
-        name: 'Booking tool',
-        amount: 99.0
-      }
-    ]
-  }
-})
-```
-
 ## `users.createCharge({userId, charge})`
 Create a charge for a given `userId` and `charge`
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "deskbookers",
-  "version": "5.2.2",
+  "version": "5.2.3",
   "description": "Deskbookers API JavaScript SDK",
   "main": "dist/index.js",
   "scripts": {

src/resources/payments/Invoices.js

@@ -110,23 +110,6 @@ export default class Users extends Resource {
     })
   }
 
-  /**
-   * Create subscription for users
-   *
-   * @param {int} userId - User Id.
-   * @param {string} token - Source token.
-   * @param {string} email - Email Source token.
-   * @param {bool} setDefault - Default flag token.
-   * @return {Object[]} - sources.
-   */
-  async createSubscription ({userId, subscription} = {}) {
-    return await this.request({
-      method: 'POST',
-      path: `${this.endpoint}/${userId}/subscriptions`,
-      body: subscription || {}
-    })
-  }
-
   /**
    * Get user souces
    *
@@ -169,7 +152,8 @@ export default class Users extends Resource {
   async setDefaultSource ({userId, sourceId} = {}) {
     return await this.request({
       method: 'PUT',
-      path: `${this.endpoint}/${userId}/defaultsource/${sourceId}`
+      path: `${this.endpoint}/${userId}/defaultsource/${sourceId}`,
+      body: { sourceId }
     })
   }
 

src/resources/payments/Plans.js

@@ -1,16 +1,10 @@
 import Resource from '../Resource'
 import Users from './Users'
-import Plans from './Plans'
-import Subscriptions from './Subscriptions'
-import Invoices from './Invoices'
 
 export default class Payments extends Resource {
   constructor (api) {
     super(api)
     // Create sub-resources
     this.users = new Users(api)
-    this.plans = new Plans(api)
-    this.subscriptions = new Subscriptions(api)
-    this.invoices = new Invoices(api)
   }
 }