Merge pull request #421 from bvandorf/master

fix account number and time in force and 400 error

robin_stocks/robinhood/helper.py

@@ -349,12 +349,11 @@ def request_post(url, payload=None, timeout=16, json=False, jsonify_data=True):
                 'Content-Type', 'application/x-www-form-urlencoded; charset=utf-8')
         else:
             res = SESSION.post(url, data=payload, timeout=timeout)
-        if res.status_code in [500,400]:
+        if res.status_code not in [200, 201, 202, 204, 301, 302, 303, 304, 307, 400, 401, 402, 403]:
             raise Exception("Received "+ str(res.status_code))
         data = res.json()
     except Exception as message:
         print("Error in request_post: {0}".format(message), file=get_output())
-    # Either return response <200,401,etc.> or the data that is returned from requests.
     if jsonify_data:
         return(data)
     else:

robin_stocks/robinhood/orders.py

@@ -901,7 +901,7 @@ def order_option_credit_spread(price, symbol, quantity, spread, timeInForce='gtc
     such as the order id, the state of order (queued, confired, filled, failed, canceled, etc.), \
     the price, and the quantity.
     """
-    return(order_option_spread("credit", price, symbol, quantity, spread, timeInForce, account_number, jsonify))
+    return(order_option_spread("credit", price, symbol, quantity, spread, account_number, timeInForce, jsonify))
 
 
 @login_required
@@ -933,7 +933,7 @@ def order_option_debit_spread(price, symbol, quantity, spread, timeInForce='gtc'
     such as the order id, the state of order (queued, confired, filled, failed, canceled, etc.), \
     the price, and the quantity.
     """
-    return(order_option_spread("debit", price, symbol, quantity, spread, timeInForce, account_number, jsonify))
+    return(order_option_spread("debit", price, symbol, quantity, spread, account_number, timeInForce, jsonify))
 
 
 @login_required