why i put it as string? fml