added my loop, but it's not working