SmartThingsCommunity / smartapp-sdk-nodejs

Javascript/NodeJS SDK to create SmartThings SmartApps
https://smartthings.developer.samsung.com/
Apache License 2.0
146 stars 80 forks source link

I18n default value text setting #126

Closed MJ17 closed 4 years ago

MJ17 commented 4 years ago

I found that the sdk does not support i18n for defaultValue of text-setting I created an issue ticket https://github.com/SmartThingsCommunity/smartapp-sdk-nodejs/issues/125 Since it's just an easy task, I changed the code by myself and created a pull request. Please take a look.

Types of changes

added a line of code in the constructor of text-setting.

lgtm-com[bot] commented 4 years ago

This pull request introduces 1 alert when merging aa8f5ef649fa69e1e1b7e7c35d533cc8fb62a972 into ea3ed0f8aac5bd0efb0cac85289895d2225dddd0 - view on LGTM.com

new alerts: