fixup! Apply suggestions from code review

Author: lpinca

README.md

@@ -482,12 +482,13 @@ default.
 - fulfillmentOrder
   - `cancel(id, params)`
   - `close(id[, message])`
+  - `fulfillments(id)`
   - `get(id)`
+  - `hold(id, params)`
   - `list([params])`
   - `locationsForMove(id)`
   - `move(id, locationId)`
   - `setFulfillmentOrdersDeadline(params)`
-  - `fulfillments(id)`
 - fulfillmentRequest
   - `accept(fulfillmentOrderId[, message])`
   - `create(fulfillmentOrderId, params)`

resources/fulfillment-order.js

@@ -131,9 +131,9 @@ FulfillmentOrder.prototype.fulfillments = function fulfillments(id) {
  * Halts all fulfillment work on a fulfillment order with
  * status OPEN and changes the status of the fulfillment order to ON_HOLD.
  *
- * @param {Number} id Fulfillment Order id (fulfillment_order_id).
-* @param {Object} params An object containing the reason for the fulfillment
-      hold and additional optional information
+ * @param {Number} id Fulfillment Order ID
+ * @param {Object} params An object containing the reason for the fulfillment
+       hold and additional optional information
  * @return {Promise} Promise that resolves with the result
  * @public
  */

test/fixtures/fulfillment-order/req/index.js

@@ -2,6 +2,6 @@
 
 exports.cancel = require('./cancel');
 exports.close = require('./close');
+exports.hold = require('./hold');
 exports.move = require('./move');
 exports.setFulfillmentOrdersDeadline = require('./set-fulfillment-orders-deadline');
-exports.hold = require('./hold');

test/fulfillment-order.test.js

@@ -143,8 +143,10 @@ describe('Shopify#fulfillmentOrder', () => {
       .post('/admin/fulfillment_orders/1046000789/hold.json', input)
       .reply(200, output);
 
-    return shopify.fulfillmentOrder.hold(1046000789).then((data) => {
-      expect(data).to.deep.equal(output.fulfillment_order);
-    });
+    return shopify.fulfillmentOrder
+      .hold(1046000789, input.fulfillment_hold)
+      .then((data) => {
+        expect(data).to.deep.equal(output.fulfillment_order);
+      });
   });
 });

types/index.d.ts

@@ -3572,6 +3572,11 @@ declare namespace Shopify {
     updated_at: string;
   }
 
+  interface IFulfillmentHoldFulfillmentOrderLineItem {
+    id: number;
+    quantity: number;
+  }
+
   interface IFulfillmentHold {
     reason:
       | 'awaiting_payment'
@@ -3581,11 +3586,6 @@ declare namespace Shopify {
       | 'other';
     reason_notes?: string;
     notify_merchant?: boolean;
-    fulfillment_order_line_items?: IUpdateFulfillmentOrderLineItem[];
-  }
-
-  interface IUpdateFulfillmentOrderLineItem {
-    id: number;
-    quantity: number;
+    fulfillment_order_line_items?: IFulfillmentHoldFulfillmentOrderLineItem[];
   }
 }