ioBroker / ioBroker.zwave

Legacy Z-Wave integration using OpenZWave
Other
24 stars 7 forks source link

Confirmation for destructive action #52

Closed cburghardt closed 5 years ago

cburghardt commented 5 years ago

I just clicked by accident on the hard reset button and was shocked that all my zwave devices where wiped out. While the function works as expected I would recommend a confirmation dialog.

cburghardt commented 5 years ago

@AlCalzone I will provide a PR for this. I am missing the i18n sources in the repository?

AlCalzone commented 5 years ago

I think there is a gulp command to generate those... but don't ask me which. I'm using those too seldom.

cburghardt commented 5 years ago

@Apollon77 do you know how that works if I need to add new strings to the admin page? The generated output is words.js in the admin directory but I have no idea how to generate that.

DutchmanNL commented 5 years ago

@cburghardt hope this helps ?

https://forum.iobroker.net/topic/19047/gulp-ist-kein-hexenwerk-auto-translation

cburghardt commented 5 years ago

@DutchmanNL awesome, that was the solution @AlCalzone PR created

cburghardt commented 5 years ago

Implemented