From 5e1fb11124cf67df36e29876698450714132a51a Mon Sep 17 00:00:00 2001
From: misagh <misagh.shakeri@gmail.com>
Date: Sun, 25 Nov 2018 17:30:06 +0100
Subject: [PATCH] documentation added for stop loss on exchange

---
 config_full.json.example       | 3 ++-
 docs/configuration.md          | 9 +++++----
 freqtrade/constants.py         | 2 +-
 freqtrade/exchange/__init__.py | 2 +-
 4 files changed, 9 insertions(+), 7 deletions(-)

diff --git a/config_full.json.example b/config_full.json.example
index b0719bcc6..6134e9cad 100644
--- a/config_full.json.example
+++ b/config_full.json.example
@@ -36,7 +36,8 @@
     "order_types": {
         "buy": "limit",
         "sell": "limit",
-        "stoploss": "market"
+        "stoploss": "market",
+        "stoploss_on_exchange": "false"
     },
     "exchange": {
         "name": "bittrex",
diff --git a/docs/configuration.md b/docs/configuration.md
index 62559a41e..03059e261 100644
--- a/docs/configuration.md
+++ b/docs/configuration.md
@@ -39,7 +39,7 @@ The table below will list all configuration parameters.
 | `ask_strategy.use_order_book` | false | No | Allows selling of open traded pair using the rates in Order Book Asks.
 | `ask_strategy.order_book_min` | 0 | No | Bot will scan from the top min to max Order Book Asks searching for a profitable rate.
 | `ask_strategy.order_book_max` | 0 | No | Bot will scan from the top min to max Order Book Asks searching for a profitable rate.
-| `order_types` | None | No | Configure order-types depending on the action (`"buy"`, `"sell"`, `"stoploss"`).
+| `order_types` | None | No | Configure order-types depending on the action (`"buy"`, `"sell"`, `"stoploss"`, `"stoploss_on_exchange"`).
 | `exchange.name` | bittrex | Yes | Name of the exchange class to use. [List below](#user-content-what-values-for-exchangename).
 | `exchange.key` | key | No | API key to use for the exchange. Only required when you are in production mode.
 | `exchange.secret` | secret | No | API secret to use for the exchange. Only required when you are in production mode.
@@ -141,17 +141,18 @@ end up paying more then would probably have been necessary.
 
 ### Understand order_types
 
-`order_types` contains a dict mapping order-types to market-types. This allows to buy using limit orders, sell using limit-orders, and create stoploss orders using market.
+`order_types` contains a dict mapping order-types to market-types as well as stoploss on or off exchange type. This allows to buy using limit orders, sell using limit-orders, and create stoploss orders using market. It also allows to set the stoploss "on exchange" which means stoploss order would be placed immediately once the buy order is fulfilled.
 This can be set in the configuration or in the strategy. Configuration overwrites strategy configurations.
 
-If this is configured, all 3 values (`"buy"`, `"sell"` and `"stoploss"`) need to be present, otherwise the bot warn about it and will fail to start.
+If this is configured, all 4 values (`"buy"`, `"sell"`, `"stoploss"`, `"stoploss_on_exchange"`) need to be present, otherwise the bot warn about it and will fail to start.
 The below is the default which is used if this is not configured in either Strategy or configuration.
 
 ``` json
     "order_types": {
         "buy": "limit",
         "sell": "limit",
-        "stoploss": "market"
+        "stoploss": "market",
+        "stoploss_on_exchange": "false"
     },
 ```
 
diff --git a/freqtrade/constants.py b/freqtrade/constants.py
index 481a219d6..86067d395 100644
--- a/freqtrade/constants.py
+++ b/freqtrade/constants.py
@@ -13,7 +13,7 @@ DEFAULT_HYPEROPT = 'DefaultHyperOpts'
 DEFAULT_DB_PROD_URL = 'sqlite:///tradesv3.sqlite'
 DEFAULT_DB_DRYRUN_URL = 'sqlite://'
 UNLIMITED_STAKE_AMOUNT = 'unlimited'
-REQUIRED_ORDERTYPES = ['buy', 'sell', 'stoploss']
+REQUIRED_ORDERTYPES = ['buy', 'sell', 'stoploss', 'stoploss_on_exchange']
 ORDERTYPE_POSSIBILITIES = ['limit', 'market']
 
 
diff --git a/freqtrade/exchange/__init__.py b/freqtrade/exchange/__init__.py
index 6e826794a..a311fd666 100644
--- a/freqtrade/exchange/__init__.py
+++ b/freqtrade/exchange/__init__.py
@@ -227,7 +227,7 @@ class Exchange(object):
                 raise OperationalException(
                     f'Exchange {self.name} does not support market orders.')
 
-        if order_types.get('stoploss_on_exchange', False):
+        if order_types.get('stoploss_on_exchange'):
             if self.name is not 'Binance':
                 raise OperationalException(
                     'On exchange stoploss is not supported for %s.' % self.name