En mi humilde opinión, es mas problema de conexiones que de programación, puedes poner el esquema de las conexiones que has realizado?
por otro lado, en el void loop(), utilizas delay para controlar el tiempo que los led permanecen encendidos, eso te para la ejecución de todo el programa, te iría mejor controlando el tiempo que transcurre en vez de pararlo (eso no tiene nada que ver con lo que comentas que te sucede)