You have wired the permanent live directly to the bulb "L", the live needs to go into the connector called "loop" (should be the middle connector block with three spaces for connections). The switch live should go to the connector connected to the bulb and one into the loop also. Hope this helps. Will