littlebits / cloud-client-api-http

Lightweight wrapper for littleBits Cloud HTTP API
GNU General Public License v3.0
31 stars 10 forks source link

device_id must be camelCased acceptable #2

Closed jasonkuhrt closed 10 years ago

jasonkuhrt commented 10 years ago

Currently path-params are not customizable so we're stuck with device_id e.g.

output({device_id: 'foo', percent: 50})

We could use deviceId but wow that would suck. We could use id or device.

I solved this type of problem before forsubscriber_id and publisher_id by using subscriber and publisher. Similarly I think I prefer device here:

output({device: 'foo', percent: 50})

If we had snake_case support in the community I wouldn't think twice about renaming these keys.

jasonkuhrt commented 10 years ago

In lieu of #4 we should actually be supporting both camelCase or snake_case.

jasonkuhrt commented 10 years ago

Resolved by #4 automatically.