I think you should also have a default value for this