fine but not working