Hi Jason.
Got it into the wall. the screws that came with the faceplate are too short so had to fudge it with some longer ones which have rounded heads which don't let me mount the faceplate properly..
But seems to work, sees all landing motion without any issues.
Just trying to get the controller to handle retriggers to keep the light on 
domoticz i don't think have plans to support this, they took out the ability to use free text variables.
I plan on doing 1 minute cycles for irrigation and using a moisture sensor to evaluate how much longer the cycles need to run for.
https://www.openhardware.io/view/4/EasyNewbie-PCB-for-MySensors
Made up in China for about £10 Inc shipping I think
Dirtypcbs is an awesome website
Hi Jason.
Got it into the wall. the screws that came with the faceplate are too short so had to fudge it with some longer ones which have rounded heads which don't let me mount the faceplate properly..
But seems to work, sees all landing motion without any issues.
Just trying to get the controller to handle retriggers to keep the light on
I'm going down the diodes route... another channel to trigger it is tricky to deal with unless you use normally off relays and use the code earlier in this thread. I tried to use it but use normally on relays so going for diodes.
K.I.S.S - keep it simple stupid 
no point making it harder for you if you have a workaround on the table
@Sergio-Rius no problem, i had plenty of issues trying to get this to work with openhab before i changed controller
Yes, i've had to manually set the valve runtimes on the ardunio code, because its not possible to pass the runtime variable etc, this is from the one i''ve got built. runtime is 1min, but you can edit as required.
else if (message.type == V_VAR1)
{
int variable1 = 60000; // atoi(message.data);// RUN_ALL_ZONES time
DEBUG_PRINT(F("Recieved variable1 valve:"));
DEBUG_PRINT(i);
DEBUG_PRINT(F(" = "));
DEBUG_PRINTLN(variable1);
if (variable1 != allZoneTime[i])
{
allZoneTime[i] = variable1;
zoneTimeUpdate = true;
}
}
else if (message.type == V_VAR2)
{
int variable2 = 60000; //atoi(message.data);// RUN_SINGLE_ZONE time
DEBUG_PRINT(F("Recieved variable2 valve:"));
DEBUG_PRINT(i);
DEBUG_PRINT(F(" = "));
DEBUG_PRINTLN(variable2);
if (variable2 != valveSoloTime[i])
{
valveSoloTime[i] = variable2;
zoneTimeUpdate = true;
}
}
Building a mysensors PIR/Temp/Light sensor for the top of my stairs, got the first part of the build done (the other sensors will be exposed through the faceplate spacer side/top.
Feel free to give me your opinions, can share the STL if required, it is a super simple shape though.. it was my first 3D printed design 
Oh, well this is strange, if i disconnect the pin to the relay, it does not activate when unpowered.
but from the shift register it is +5 when the zone is not active, and 0V when active, so i assume its an active low not active high by your example... surely removing the 5V feed to the relay pin should make it activate.. not sure whats going on there, some sort of black magic 
My reading of shift registers has made my brain hurt.. do i need to reverse the 0000000010000000 for 1111111101111111?
@BulldogLowell
Hi!
Rather than using valve 8, i am using only a block of 4, but currently have a 2 channel for testing.
My reading into shift registers showed 0b0000000010000000 is 2x 8 bit outputs, B0-B7
tried configuing B0 and B7 to try and activate relay 1, was still seeing 5V on all pins apart from the output pin for the zone in use (EG zone 2 expecting Zone1 to alway be on)
or am i doing this completely wrong?
Gave up with using openhab now using domoticz which is a lot nicer, forced it to hard coded run times by doing this:
else if (message.type == V_VAR1)
{
int variable1 = 5; // atoi(message.data);// RUN_ALL_ZONES time
DEBUG_PRINT(F("Recieved variable1 valve:"));
DEBUG_PRINT(i);
DEBUG_PRINT(F(" = "));
DEBUG_PRINTLN(variable1);
if (variable1 != allZoneTime[i])
{
allZoneTime[i] = variable1;
zoneTimeUpdate = true;
}
}
else if (message.type == V_VAR2)
{
int variable2 = 5; //atoi(message.data);// RUN_SINGLE_ZONE time
DEBUG_PRINT(F("Recieved variable2 valve:"));
DEBUG_PRINT(i);
DEBUG_PRINT(F(" = "));
DEBUG_PRINTLN(variable2);
if (variable2 != valveSoloTime[i])
{
valveSoloTime[i] = variable2;
zoneTimeUpdate = true;
}```
one thing to note the shift register 0b0000000010000000 does not seem to work, will work out a plan around this as i need to run a 12 pump off the relay bank to draw water from the water butt.