The Date Popup module is missing a README, which serves as documentation.

Added a merge request to add a README.md to Date Popup.
Kindly review.

@gslente Thank you for adding the README.md file. However, there seems to be a typo in the filename. It is currently named READE.md instead of README.md. Please rename it accordingly.

rushikesh raval → made their first commit to this issue’s fork.

Hi,

Thank you for providing an MR. I applied MR!13, it was applied smoothly and provided a README.md file.

Just a few things to change. According to the README.md template → ,

1. Two lines prior to ##/### headings, in the current, there are one.
2. In the table of contents, I think "-" is better than the numbered list
3. Ordered lists use "1", in the current, it is incrementing

Please check and advise since I am still new to Drupal.
Thank you.

Updated as per comment #7

Hi @rushikesh raval,

Thank you for updating the MR. I applied MR!13, it was applied smoothly and the changes were reflected.

➜  date_popup git:(2.0.2) curl https://git.drupalcode.org/project/date_popup/-/merge_requests/13.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   956    0   956    0     0   2146      0 --:--:-- --:--:-- --:--:--  2148
patching file README.md
➜  date_popup git:(2.0.2) ✗

Just some clarification, in the Installation section, is that supposed to be an ordered list? My understanding is that those two are steps.

If it is indeed an ordered list, then kindly use the "1" instead of the "-", please see README.md template → .

Otherwise, this is good to go for the maintainers to review.

Moving the status to Needs work for the meantime.
Thank you.

@cleavinjosh Thanks for your review. Please check README.md template → . There is no firm rules for ordered list / unordered list.