- Issue created by @sonam_sharma
- Issue was unassigned.
- Status changed to Needs work
about 1 year ago 5:21pm 27 June 2023 - 🇵🇭Philippines kenyoOwen
Hi sonam_sharma
I reviewed your patch and it is not following the README.md template → and your patch doesn't apply. README.txt should be removed and replaced with README.md hence it will be redundant. Please see the screenshots attached.
Thank you.
- @sonam_sharma opened merge request.
- 🇮🇳India Anjali Mehta
Anjali Mehta → made their first commit to this issue’s fork.
- Status changed to Needs review
12 months ago 7:52am 28 June 2023 - 🇮🇳India Anjali Mehta
Hello Everyone
Updated the file. Please review the changes.
Thank You. @Anjali Mehta, Verified the MR8 and the readme.md is updated as per standard. Added the screenshots for reference.
The only issue I can see the patch is not cleanly applied ->
8.patch:72: new blank line at EOF.
+
warning: 1 line adds whitespace errors.Please have a look.
- First commit to issue fork.
- Status changed to Needs work
12 months ago 10:32am 3 July 2023 - 🇮🇹Italy apaderno Brescia, 🇮🇹
The issue summary should always describe what should be fixed/changed, even in the case the issue is blatantly obvious. Neither the title nor screenshots are sufficient to describe what the issue is, even in the case a patch is provided.
- First commit to issue fork.
- Status changed to Needs review
12 months ago 7:13am 4 July 2023 - Status changed to RTBC
12 months ago 6:18pm 6 July 2023 - 🇮🇳India Bushra Shaikh
Verified the MR !8 and the readme.md is updated as per standard. Added the screenshots for reference.
Can be move to RTBC +1
- First commit to issue fork.
-
arti_parmar →
committed e16cd4da on 1.0.x authored by
sonam_sharma →
Issue #3370593: missing Readme.md file
-
arti_parmar →
committed e16cd4da on 1.0.x authored by
sonam_sharma →
- Status changed to Fixed
11 months ago 10:24am 19 July 2023 Automatically closed - issue fixed for 2 weeks with no activity.