Yeah, seems like a good thing to include. I think with doing something like that we should make a start on returning objects instead of dictionaries too? Thoughts?
Evaldo Bratti
@evaldobratti
I don't know if there is a lot of changes we can make from the json's objects, we could just make a python object equivalent to the json object, se we can access fields in a object way instead of map way. That would be good I think. The problem is to parse the response to correct objects :worried:
Joshua Duffy
@joshuaduffy
Yeah will be a lot of work, I've made a start on it however :) Should take a few weeks but the response object will be either a Match, or a list of matches, a user etc.
then we can have objects like heroes & players within the match
Evaldo Bratti
@evaldobratti
Just pushed a branch with parser of a match
Currently it's just parsing a match if the root of the result is a match with all informations
there are tests failing because of that
but it's just a start
maybe we can work on the same branch
Evaldo Bratti
@evaldobratti
get match and get match history are parsed, I think
there are the other responses, as get items, get heroes, leagues etc
Evaldo Bratti
@evaldobratti
I'll work again during the week
Joshua Duffy
@joshuaduffy
Hey, same, working lates at the moment, so not much time, but hopefully next week I will have :)