- Issue created by @Rashmi Gaurana
- @rashmi-gaurana opened merge request.
- Issue was unassigned.
- Status changed to Needs review
almost 2 years ago 1:48pm 4 February 2023 - 🇮🇳India nayana_mvr
Verified the MR!6 and tested it on Drupal version 10.1.x and Welcome mail version 8.x-1.x. The patch applied cleanly but there was a warning message wile applying the patch:
6.diff:25: trailing whitespace. Checking patch README.md... Checking patch README.txt... Applied patch README.md cleanly. Applied patch README.txt cleanly. warning: 1 line adds whitespace errors.
Few more changes required:-
As per README.md template → ,
-Bulleted lists should be denoted by dashes(-)
-Nested lists should be indented with 4 spaces - First commit to issue fork.
- First commit to issue fork.
- Merge request !7Issue #3339234: Replace README.txt with README.md file → (Merged) created by royalpinto007
- 🇮🇳India royalpinto007
Hello,
I would like to inform you that I have made all the required changes in MR #7. I hope you find the modifications satisfactory.
Could you please take the time to review the changes and provide your feedback?
I would greatly appreciate it. - 🇮🇳India nayana_mvr
There were some coding standard issues. Fixed that as well in the new commit.
- Assigned to sourabhjain
- Issue was unassigned.
- Status changed to RTBC
over 1 year ago 7:36am 20 February 2023 - 🇮🇳India sourabhjain
I have reviewed the MR7 and its looks fine. Moving to RTBC.
- Assigned to eiriksm
- last update
10 months ago 3 pass -
eiriksm →
committed 3e7fd196 on 8.x-1.x authored by
royalpinto007 →
Issue #3339234 by Rashmi Gaurana, nayana_mvr, eiriksm: Replace README....
-
eiriksm →
committed 3e7fd196 on 8.x-1.x authored by
royalpinto007 →
- Issue was unassigned.
- Status changed to Fixed
10 months ago 8:10pm 14 January 2024 Automatically closed - issue fixed for 2 weeks with no activity.