shld be string