Account created on 12 December 2017, about 7 years ago
#

Merge Requests

More

Recent comments

🇮🇳India vimal_nadar

I installed the Paragraphs Bootstrap Carousel module, added the field to a content type, and successfully uploaded an image without encountering the previously mentioned error. Please see the attached screenshot below.

🇮🇳India vimal_nadar

I have created MR !44. Please review it.

🇮🇳India vimal_nadar

I have created MR43. Please review it.

🇮🇳India vimal_nadar

I have created MR5. Please review it.

🇮🇳India vimal_nadar

@Raveen Thakur, why do we require a patch if we have already created MR ? If everything is fine in my MR, you could have moved my ticket to RTBC ?

🇮🇳India vimal_nadar

@Raveen Thakur, why do we require a patch if we have already created MR ?

🇮🇳India vimal_nadar

I have created MR13. Please review it.

🇮🇳India vimal_nadar

I have created MR3. Please review it.

🇮🇳India vimal_nadar

I have created MR. Please review it.

🇮🇳India vimal_nadar

I have created MR. Please review it.

🇮🇳India vimal_nadar

@kenyoOwen, Updated the code. You can review it.

🇮🇳India vimal_nadar

@kenyoOwen, the numbering is correct only. If we give number 1 to all the points it will auto convert into the sequence.
Attaching the output of the README.md for reference.

🇮🇳India vimal_nadar

@clarkssquared, I have added more info to the hook_help. Please review it.

🇮🇳India vimal_nadar

I have created the MR. Please review it.

🇮🇳India vimal_nadar

I have created MR. Please review it.

🇮🇳India vimal_nadar

@Raveen Thakur, this ticket is already in RTBC, was there a need to test it again ?

🇮🇳India vimal_nadar

I have created a MR. Please review it.

🇮🇳India vimal_nadar

@Neslee Canil Pinto, Are there no credits for the reviewers?

🇮🇳India vimal_nadar

I have created a MR. Please review it.

🇮🇳India vimal_nadar

I have reviewed the MR 1 and everything looks fine according to the Drupal standards.
Moving it to RTBC.

🇮🇳India vimal_nadar

I have created the MR. Please review it.

🇮🇳India vimal_nadar

I have reviewed the MR 3 and everything looks fine according to the Drupal standards.
Moving it to RTBC.

🇮🇳India vimal_nadar

I have created MR. Please review it.

🇮🇳India vimal_nadar

I have created MR. Please review it.

🇮🇳India vimal_nadar

I have created MR. Please review it.

🇮🇳India vimal_nadar

Created MR. Please review it.

🇮🇳India vimal_nadar

@Raghavendra A M, I looked into your MR and found that "Project Name" is missing from the top of the README.md file. Everything else looks good. Moving it to Needs work.

🇮🇳India vimal_nadar

Created MR. Please review it.

🇮🇳India vimal_nadar

Created MR. Please review it

🇮🇳India vimal_nadar

I have created MR. Please review it.

🇮🇳India vimal_nadar

Created a MR request. Please review it.

🇮🇳India vimal_nadar

Created a MR request. Please review it.

🇮🇳India vimal_nadar

@dharmeshbarot I have reviewed MR 6 and it satisfies all the Drupal standards for README.md file.

Moving it to RTBC.

🇮🇳India vimal_nadar

I have reviewed the MR 9 and everything looks fine according to the Drupal standards.
Moving it to RTBC.

🇮🇳India vimal_nadar

@akshaydalvi212 Project name in the top of the README.md should be mentioned in single # you have used ## for that. Other than that everything looks fine.

For your reference https://www.drupal.org/docs/develop/managing-a-drupalorg-theme-module-or...

🇮🇳India vimal_nadar

@clarkssquared i have removed README.txt file and kept only README.md file. You can review the changes now.

Thanks.

🇮🇳India vimal_nadar

@bandanasharma we need to add Project name in the top of the README.md file which is missing. And #Requirements section is also missing in the file.

You can refer this link https://www.drupal.org/docs/develop/managing-a-drupalorg-theme-module-or... to see which all sections is required.

🇮🇳India vimal_nadar

@Raghavendra A M I reviewed MR 9 and it does not satisfies all formatting suggested in https://www.drupal.org/docs/develop/managing-a-drupalorg-theme-module-or...
There is Project name and Requirements missing in the README.md file which is a required sections.

🇮🇳India vimal_nadar

I reviewed MR 1 and it satisfies all formatting suggested in https://www.drupal.org/docs/develop/managing-a-drupalorg-theme-module-or ...

Moving it to RTBC.

🇮🇳India vimal_nadar

@Ranjit1032002 ##Introduction is missing in the raised MR

🇮🇳India vimal_nadar

@_pratik_ the MR raised does not satisfies all formatting suggested in https://www.drupal.org/docs/develop/managing-a-drupalorg-theme-module-or...
You need to add #Requirements which is a required section in README.md file.
Also all the table of contents heading should start with ## but your table of contents heading has only one #.

🇮🇳India vimal_nadar

@_pratik_ you have not updated the status to needs review I will update the status and review your MR

🇮🇳India vimal_nadar

I will work on this.

🇮🇳India vimal_nadar

Created a MR request. Please review it.

🇮🇳India vimal_nadar

Created a MR request. Please review it.

🇮🇳India vimal_nadar

Created a MR request. Please review it.

Production build 0.71.5 2024