OK - feedback
Not a criticism and not sure as it is really an improvement, but there are two comments that spring to mind from a quick look:
1. You've used constants and one define. Define is out of favour and personally I'd change the one define to just be another constant
2. You have several relay toggle functions. The only difference between them is the relay they toggle (ie which pin) and the text they display (don't think I there were any other differences?). You could do that with just one function and pass in the text to be displayed and pin number as parameters. Would reduce the amount of code.
As I say, arguably not improvements, but features which are out of favour can sometimes be removed from compiler support (not likely in the near future) and changing to pass the parameters into the relay toggle function would mean adding additional relays would just need an additional call to the same function rather than needing a new function. I doubt your sketch is anywhere near storage limits so not really much of an advantage in this case, but (IMO) makes the code a bit simpler
Not to suggest what you have done is wrong. Neither of those changes would actually affect the way it works at all.
Tim
