should this be a string?