Merge pull request #4 from phillipivan/main

Hide redundant controller config fields when not required

companion/HELP.md

@@ -30,6 +30,9 @@ The alarms should match the number of configured alarm indexes in your system. R
 
 ## Version History
 
+### Version 1.0.4
+- Hide redundant config fields when not required
+
 ### Version 1.0.3
 - Better status updates
 - Update dependencies

package.json

@@ -1,6 +1,6 @@
 {
 	"name": "ntp-technology-dot",
-	"version": "1.0.3",
+	"version": "1.0.4",
 	"main": "src/main.js",
 	"scripts": {
 		"test": "echo \"Error: no test specified\" && exit 1",

src/config.js

@@ -49,6 +49,19 @@ module.exports = {
 				default: '',
 				width: 8,
 				regex: Regex.HOSTNAME,
+				isVisible: (options) => {
+					return options.redundant
+				},
+			},
+			{
+				type: 'static-text',
+				id: 'place-holder-1',
+				label: '',
+				value: '',
+				width: 8,
+				isVisible: (options) => {
+					return !options.redundant
+				},
 			},
 			{
 				type: 'number',
@@ -59,12 +72,25 @@ module.exports = {
 				min: 1,
 				max: 65535,
 				tooltip: 'Default: 10005',
+				isVisible: (options) => {
+					return options.redundant
+				},
+			},
+			{
+				type: 'static-text',
+				id: 'place-holder-2',
+				label: '',
+				value: '',
+				width: 4,
+				isVisible: (options) => {
+					return !options.redundant
+				},
 			},
 			{
 				type: 'checkbox',
 				id: 'redundant',
 				label: 'Redundant Controllers',
-				default: true,
+				default: false,
 				width: 4,
 			},
 			{

src/main.js

@@ -18,8 +18,8 @@ class NTP_DOT_PROTOCOL extends InstanceBase {
 		this.updateStatus(InstanceStatus.Connecting)
 		this.config = config
 		if (this.config.redundant) {
-			if (this.config.hostSec == '' || this.config.portSec == '') {
-				this.log('error', 'Secondary host / port not defined')
+			if (this.config.hostSec == '') {
+				this.log('error', 'Secondary host not defined')
 				this.updateStatus(InstanceStatus.BadConfig, 'Secondary not defined')
 				return undefined
 			}