Shouldn't each LED have its own current limiting resistor instead of wiring them in parallel with one resistor in series?
From my (way past) days as EE I remember that LEDs at least used to have sufficiently variable threshold voltages such that a circuit like yours would typically result in markedly variable light output between the LEDs. (Explanation: The LED with the lowest threshold voltage draws much higher current than the others due to the non-linear characteristics of current over voltage at the threshold.) Worst case scenario is that this LED gets driven past its rated power and burns up. Then the same repeats with the LED with the next lowest threshold ...
I don't pretend to know how the LEDs you chose will behave or that there hasn't been improvement in the technology or variability to make it work as you designed, but I wanted to raise the question after I saw your circuit board, before you finalize the design.