shouldnt be using strings as ids