Please add a requirements section.
After installing the module, the user sees there is a dependency, written as: Date_popup (missing)
But no module called Date_popup exists. It's in fact under Date, but this isn't very clear. It would help to have the dependency on Date explicitly listed.