Controlling multiple timed relays off buttons

Read How to get the best out of this forum and post your code using code tags </>.

If you properly format your code you will see you have an extra closing bracket before the switch statement. Also, case matters for variables.

I would suggest you look at the following tutorial Example-code for timing based on millis()