still not working correclty