Add support for API version 3 - #13
Conversation
There was a problem hiding this comment.
Actually, in v3 you can request a flat list:
There was a problem hiding this comment.
Oh shiny, that wasn't there yesterday when I was double-checking this. Well in that case 95% of the messy stuff can be deleted.
|
@jbt You should've left a comment when you updated this. There are no notifications for new commits, so I thought this was still waiting on you... I'll review this soon. |
|
Oops, sorry! |
|
Can you update the docs for the new data structure in v3? |
I think this covers everything.
|
There we go, think that's everything updated in the readme now. |
There was a problem hiding this comment.
May want to add "(null for desktop browsers)" to the description of device as well.
|
There are issues with this implementation which were brought up by @Krinkle and not addressed. |
Sorry it's a bit messy with the triple-nested map, but I've tried to keep it in concordance with the existing coding style. This seems to work whenever I test it.