not working - needs changing the action fuction