Please format your code correctly: see here.
The switches are showing 'ON' or 'OFF' — why are you using "Open" and "Closed" in your sensor config? payload_on needs to be whatever the MQTT message (out of value_template) is for the switch to be on. Should be:
payload_on: "ON"
payload_off: "OFF"
…and I suspect one of those will need reversing as the switches are showing different things, open or closed.
I assume there’s a top-level mqtt: declaration ahead of the first line you’ve pasted?