Turn Off Light when switch is On

Configuration
1

Hi All
I have a Sonoff Mini that is conected to a Projector screen relay. When the Projector is turned on, The Sonoff Mini is displayed as On.
I have a MiLight which is synced to 8 GU10 Lamps.
In my Home Assistant, I can see the status of the switch from On to Off depending on the Projector.
I can also from the Overview turn the Lights On/Off or change colour.
In the automation I have set this up, thinking the For is 1 second, but not really bothered about that, but it does not work.
Any pointers please?
Thank You
Paul.

alias: Projector
description: ''
trigger:
  - platform: state
    entity_id: switch.sonoff_1001042472
    from: 'off'
    to: 'on'
    for: '1'
condition: []
action:
  - condition: state
    entity_id: light.cinema
    state: 'off'
mode: restart
2 
action:
  - service: light.turn_off
    entity_id: light.cinema
mode: restart

You might want to read up on conditions and services in the documentation. Conditions don’t change things. Services do.

3

Cheers pal.
It was all rush to experiment but will check them out
Thank you