SmartThingsCommunity/smartapp-sdk-nodejs

Setting multiple permissions is a bit confusing

Closed this issue ยท 1 comments

Is your feature request related to a problem? Please describe.
When setting a singular permissions, it's obvious and just works, but when you have to set multiple, it's easy to get tripped up.

VALID

smartapp.permissions('r:devices:*')`

INVALID

let perms = 'r:devices:* x:devices:*'
smartapp.permissions(perms)

VALID

let perms = 'r:devices:* x:devices:*'
smartapp.permissions(perms.split(' '))

Describe the solution you'd like
Analyze the incoming permissions string, object, or array and transform it intelligently.

Describe alternatives you've considered
Documentation โ€“ make it clear that we can support both a single permissions via string, or a strain array.

Additional context
When you do this wrong, it is very difficult to see why configuration doesn't work. Strongman will error out and you can figure it out with DevWorkspace's Live Logging but it honestly didn't even occur to me to look there for far too long. ๐Ÿ™„

๐ŸŽ‰ This issue has been resolved in version 1.1.2 ๐ŸŽ‰

The release is available on:

Your semantic-release bot ๐Ÿ“ฆ๐Ÿš€