DigitalPin toggle error

[PeteKnight]

First of all, there are some errors in the comments regarding pin numbering…

@Pu00112

#define RelayPin1 16  //D0
#define RelayPin2 5 //D1
#define RelayPin3 4 //D2
#define RelayPin4 0 //D3
#define RelayPin5 2  //D0   << Incorrect - GPIO2 = Pin D4 and the built-in LED
#define RelayPin6 14 //D1   << Incorrect - GPIO14 = Pin D5
#define RelayPin7 12 //D2   << Incorrect - GPIO12 = Pin D6
#define RelayPin8 13 //D3   << Incorrect - GPIO13 = Pin D7

If you are using one of the common NodeMCUs that has only one onboard LED then this is connected to GPIO2, which you are using a relay output.
If it’s a NodeMCU with two LEDs then the second on is attached to GPIO16, which is also in use as a relay output.

Which one is treated as LED_BUILTIN will depend on the option you chose when you compiled your code in the IDE…

You should also read this, about the suitability of the pins you are using…

ESP8266 GPIO pins info, restrictions and features

hi blynkers! the esp8266 mcu is widely used in iot projects. however, most gpio pins on this mcu have special / multiple functionality and are not safe to use in all circumstances. these “features” are still not well documented and not explained in a friendly manner for newbies. i see lots of cases when the “issues” are simply caused by not knowing which pins to use in which case. [esp8266 pin features] here is a basic table for startup. i’ve collected the info from various sources on the i…

As you’ll see, the NodeMCU doesn’t have enough suitable pins for your project, and you’d be better using an ESP32.

Pete.