do we want to change this from a string to a list?