Apply suggestions from code review

Author: lpinca

resources/fulfillment-order.js

@@ -132,19 +132,9 @@ FulfillmentOrder.prototype.fulfillments = function fulfillments(id) {
  * 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
- * @param {String} params.reason 'awaiting_paymet' | 'high_risk_of_fraud'
- *     | 'incorrect_address' | 'inventory_out_of_stock' | 'other'.
- * @param {String} params.reason_notes (optiona) Additional information about
- *     the fulfillment hold reason.
- * @param {Boolean} params.notify_merchant (optional) Whether the merchant
- *     should receive a notification about the fulfillment hold.
- *     If set to true, then the merchant will be notified on the Shopify
- *     mobile app (if they use it to manage their store).
- *     The default value is false.
- * @param {Array} params.fulfillment_order_line_items (optional)  array of
- *     fulfillment order line item ids and the quantity of each to move.
- * @return {Promise} Promise that resolves with the result.
+* @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
  */
 FulfillmentOrder.prototype.hold = function hold(id, params) {

types/index.d.ts

@@ -391,7 +391,7 @@ declare class Shopify {
     ) => Promise<Shopify.IPaginatedResult<Shopify.IFulfillment>>;
     hold: (
       id: number,
-      params: Shopify.IHoldFulfillmentOrder
+      params: Shopify.IFulfillmentHold
     ) => Promise<Shopify.IFulfillmentOrder>;
   };
   fulfillmentRequest: {
@@ -3572,17 +3572,16 @@ declare namespace Shopify {
     updated_at: string;
   }
 
-  interface IHoldFulfillmentOrder {
+  interface IFulfillmentHold {
     reason:
       | 'awaiting_payment'
       | 'high_risk_of_fraud'
       | 'incorrect_address'
       | 'inventory_out_of_stock'
       | 'other';
     reason_notes?: string;
-    notify_merchant: boolean;
-    line_items: IUpdateFulfillmentOrderLineItem;
-    fulfillment_order_id: number;
+    notify_merchant?: boolean;
+    fulfillment_order_line_items?: IUpdateFulfillmentOrderLineItem[];
   }
 
   interface IUpdateFulfillmentOrderLineItem {