@ gowthamrajkrishnasamy
The patch does not apply to 8.x-1.x
- Issue was unassigned.
- Status changed to Needs work
about 1 year ago 1:25pm 23 June 2023 - 🇵🇭Philippines kenyoOwen
Hi gowthamrajkrishnasamy
I reviewed your patch and it is not following the README.md template → . And your patch doesn't apply in version 8.x-1.x-dev. Please see the screenshots attached.
Thank you.
- 🇮🇳India kbk1992 Hyderabad
bharath-kondeti → made their first commit to this issue’s fork.
- @bharath-kondeti opened merge request.
- Status changed to Needs review
12 months ago 5:17am 3 July 2023 - Status changed to Needs work
12 months ago 5:41am 3 July 2023 - 🇮🇳India D-XPERT
Marking from NR to NW as the current changes don't follow the Drupal README.md Template → Standards.
Drupal recommends the following README sections:
Project name and introduction (required)
Table of contents (optional)
Requirements (required)
Recommended modules (optional)
Installation (required, unless a separate INSTALL.md is provided)
Configuration (required)
Troubleshooting & FAQ (optional)
Maintainers (optional) - Assigned to nitin_lama
- Issue was unassigned.
- Status changed to Needs review
12 months ago 8:15am 3 July 2023 - Status changed to RTBC
12 months ago 10:22am 3 July 2023 - 🇮🇳India Bushra Shaikh
I verified and applied MR !4 for Drupal version 9.5.8
README file updated to README.md after applying this patch.can be move to RTBC+1
- Status changed to Needs work
12 months ago 11:07am 3 July 2023 - 🇮🇳India D-XPERT
Hello @Nitin
There is a minor mistake in the heading .
The project name is the only level one heading in the document.
## Permanent Entities:. change it to # Permanent Entities. - Status changed to Needs review
12 months ago 11:44am 3 July 2023 - Status changed to RTBC
12 months ago 12:15pm 3 July 2023 - 🇮🇳India Anmol_Specbee
Patch in merge!4 seems to be working fine. Moving to RTBC.