From 25ee847ad05a5f7db613b4f2b3bad0221a3d4725 Mon Sep 17 00:00:00 2001
From: Ricardo Gama <rgama@uphold.com>
Date: Wed, 31 May 2017 17:46:29 +0100
Subject: [PATCH] Add destination argument to createCardTransation action
 method

---
 .../card-transaction/create-card-transaction.md     | 13 +++++++------
 src/core/actions/card-transaction.js                |  3 ++-
 test/core/actions/card-transaction.spec.js          |  6 ++++--
 3 files changed, 13 insertions(+), 9 deletions(-)

diff --git a/docs/actions/card-transaction/create-card-transaction.md b/docs/actions/card-transaction/create-card-transaction.md
index 35bc28c..4a427f3 100644
--- a/docs/actions/card-transaction/create-card-transaction.md
+++ b/docs/actions/card-transaction/create-card-transaction.md
@@ -16,12 +16,13 @@ The `commit` argument will add the `commit` query parameter.
 
 The `body` argument accepts the following keys:
 
-| Key            | Type   | Description                                                  |
-|:---------------|:-------|:-------------------------------------------------------------|
-| `amount`       | String | The value amount to send in the denominated currency         |
-| `currency`     | String | The currency by which you wish to denominate the transaction |
-| `message`      | String | Transaction details                                          |
-| `securityCode` | String | Credit card security code                                    |
+| Key            | Type   | Description                                                                                                                                                 |
+|:---------------|:-------|:------------------------------------------------------------------------------------------------------------------------------------------------------------|
+| `amount`       | String | The value amount to send in the denominated currency                                                                                                        |
+| `currency`     | String | The currency by which you wish to denominate the transaction                                                                                                |
+| `destination`  | String | The destination of the transaction, which can be in the form of a bitcoin address, an email address, an account id, an application id or an Uphold username |
+| `message`      | String | Transaction details                                                                                                                                         |
+| `securityCode` | String | Credit card security code                                                                                                                                   |
 
 This method returns a **Promise**.
 
diff --git a/src/core/actions/card-transaction.js b/src/core/actions/card-transaction.js
index e630735..080b3bb 100644
--- a/src/core/actions/card-transaction.js
+++ b/src/core/actions/card-transaction.js
@@ -28,13 +28,14 @@ export function commitCardTransaction(cardId, transactionId, { message, security
 }
 
 // eslint-disable-next-line max-params
-export function createCardTransaction(cardId, { amount, currency, message, securityCode }, commit, otp, options) {
+export function createCardTransaction(cardId, { amount, currency, destination, message, securityCode }, commit, otp, options) {
   options = merge({
     body: {
       denomination: {
         amount,
         currency
       },
+      destination,
       message,
       securityCode
     },
diff --git a/test/core/actions/card-transaction.spec.js b/test/core/actions/card-transaction.spec.js
index 629e050..5fa4426 100644
--- a/test/core/actions/card-transaction.spec.js
+++ b/test/core/actions/card-transaction.spec.js
@@ -55,7 +55,7 @@ describe('CardTransactionActions', () => {
 
   describe('createCardTransaction()', () => {
     it('should make a request to `POST /me/cards/:cardId/transactions`', () => {
-      return sdk.createCardTransaction('bar', { amount: 'biz', currency: 'baz', message: 'buz', securityCode: 'bez' }, false, false, { qux: 'qix' })
+      return sdk.createCardTransaction('bar', { amount: 'biz', currency: 'baz', destination: 'qax', message: 'buz', securityCode: 'bez' }, false, false, { qux: 'qix' })
         .then(result => {
           expect(result).toBe('foo');
           expect(sdk.api).toBeCalledWith('/me/cards/bar/transactions', {
@@ -64,6 +64,7 @@ describe('CardTransactionActions', () => {
                 amount: 'biz',
                 currency: 'baz'
               },
+              destination: 'qax',
               message: 'buz',
               securityCode: 'bez'
             },
@@ -74,7 +75,7 @@ describe('CardTransactionActions', () => {
     });
 
     it('should make a request to `POST /me/cards/:cardId/transactions` with commit query paramameter', () => {
-      return sdk.createCardTransaction('bar', { amount: 'biz', currency: 'baz', message: 'buz', securityCode: 'bez' }, true, false, { queryParams: { qux: 'qix' } })
+      return sdk.createCardTransaction('bar', { amount: 'biz', currency: 'baz', destination: 'qax', message: 'buz', securityCode: 'bez' }, true, false, { queryParams: { qux: 'qix' } })
         .then(result => {
           expect(result).toBe('foo');
           expect(sdk.api).toBeCalledWith('/me/cards/bar/transactions', {
@@ -83,6 +84,7 @@ describe('CardTransactionActions', () => {
                 amount: 'biz',
                 currency: 'baz'
               },
+              destination: 'qax',
               message: 'buz',
               securityCode: 'bez'
             },