priya_degwekar → made their first commit to this issue’s fork.
@all, Thanks for the contribution.
Hi.. I've reviewed and tested your patch. Your patch is working fine.
Please check screenshot.
Can be moved this to RTBC.
Thanks!
@chetan 11, Please clarify this point then we can move this issue to RTBC.
@chetan 11,
FYI.. We can download the commit as patch file.
So The latest commit which is in MR2, I've downloaded as patch file.
So I am talking about that patch or you can say MR2.
Also, I am not saying that MR2 is incorrect. It is fixing the issue.
I am talking about the warning which I got when applying the Patch/MR2.
After confirmation of the warning, we can move this issue to RTBC.
Issue is fixed. Please look into the warning thing.
Please check the attached screenshots.
Thanks
Hi @shweta__sharma,
I've reviewed your patch. It is applying but giving warning at the same time.
Please check the attached scrrenshot.
Please check the attached screenshots and look into this.
Thanks.
Hi @viren18febS,
I've reviewed your patch. It is applying but giving warning at the same time.
Also when I check the dropdowns, its still overlapping.
Priya_Degwekar → created an issue.
Hi,
I've created a patch to fix the issue.
Please review.
Thanks!
Hi,
I have created a patch to resolve the issue
#12
📌
Add a README file
Needs work
.
Please review.
thanks
Hi, I've fixed the issue mentioned #10 📌 Update README.md file according to "README.md template" Needs review in this commit
Please review.
Thanks.
Priya_Degwekar → made their first commit to this issue’s fork.
Hi, I've fixed the maintainer section in this commit
Please review.. thanks!
Priya_Degwekar → made their first commit to this issue’s fork.
Hi @nitin_lama,
I have reviewed and applied your MR. It is following the
README.md template →
A minor issue is that First maintainer has not mentioned his name in the profile, He has only displaying his username. It's better we should not use the name without knowing.
Thanks
Hi, @dawadorjidd, I think characters are not exceeding more than 80col at my end.
Please check the screenshots.
If you can able to correct it, Please do the corrections.
Thanks.
Ok.. I will work on it.
I have created a patch.
Please review.
Priya_Degwekar → created an issue.
Priya_Degwekar → created an issue. See original summary → .
Hi, I have created a patch which updates readme file.
Please review.. Thanks.
Priya_Degwekar → made their first commit to this issue’s fork.
Hi, I have created a patch to update readme file.
Please review, thanks!
Priya_Degwekar → created an issue.
Hi chancenyasulu → , Thanks for your suggestion, I've updated the feature section with your suggestion and opened #MR2.
Hi, I've created a patch to solve this issue.
Please review.. thanks!
Priya_Degwekar → created an issue.
Updated the patch.
Please review.. thanks!
Hi
dawadorjidd →
, Thanks for the patch.
I have reviewed your
patch
🐛
Update read me file.
Fixed
Installation section is incorrect, you have used different theme name there.
Priya_Degwekar → made their first commit to this issue’s fork.
I've created a patch.
Please review.
Priya_Degwekar → created an issue. See original summary → .
Hi,
Please review #MR5.
Thanks!
Priya_Degwekar → created an issue.
Hi Maintainer, Pls check 3372515-1.patch only as I have noticed some mistakes in earlier MR and Patch.
Thanks.
Updated the changes.
Please review the commit.
Thanks.
Updated the patch.
Check this file below readme.patch
Please review the updated patch.
Hi @apaderno, thanks for the corrections.
I've worked accordingly and created a patch.
Please review.
thanks.
ok @kenyoOwen, I am working on it.
Hi @kenyoOwen, I am following the
README.md template →
.
Can you please mention what are the points I am not following. I will work on it.
Thanks.
Hi @kenyoOwen, I have done the changes which I think was incorrect.
Please review again and let me know what are the points I am not following. I will check it.
Thanks.
Attaching screenshots for reference.
Can be move to RTBC.
Hi @Harshita mehna, I have reviewed and tested #MR4.
It is working fine.
I have opened #MR1 to fix the issue.
Please review.
thanks.
Priya_Degwekar → created an issue.
Hi, I have created a patch to fix the issue.
please review.
thanks
Priya_Degwekar → created an issue.
Priya_Degwekar → created an issue.
Attaching the screenshot.
Can be moved to RTBC.
Hi @shivam_tiwari, I have reviewed and tested #MR1. It looks good.
Hi @apaderno, Thanks for the review.
I have updated the changes in this commit
Thanks.
Patch #3 looks good.
Can be moved to RTBC.
I have created a patch.
Please review.
Thanks.
Priya_Degwekar → created an issue.
Hi,
I have created a patch to fix the issue.
Please review.
thanks.
Priya_Degwekar → created an issue.