Return concrete structs instead of json::Value

[joeirimpan]

No description provided.

[suyashb95]

@joeirimpan I'd like to work on this

[joeirimpan]

@suyash458 Its yours then :)

[suyashb95]

@joeirimpan what are your thoughts on having the Order modification/cancellation APIs as methods on the structs themselves instead of the KiteConnect client?

[joeirimpan]

I had similar thoughts. Instead of client.place_order, we could write something like client.order.place? But this will make it inconsistent with other libs that we have in place.

[suyashb95]

@joeirimpan I was thinking the APIs to fetch/place orders could be offered by the client but, the APIs to modify/cancel the orders themselves could be methods on the Order struct. So we could write something like below

let order = client.place_order();
order.cancel();

Although, as you said this doesn't make much sense if it's inconsistent with the other libraries