Each LED should have its own series current limiting resistor.
What you drew might work, but will not be the best you can do and is def not officially correct.
See
https://www.allaboutcircuits.com/technical-articles/led-arrays-one-resistor-or-many/
and
There is no input to read the pushbutton...
a7
2 Likes