Shows the total number of replies to a comment as an integer (as a string appended to 'reply' link of a comment) with a text 'Show/Hide' . On Clicking on that text the display of replies of that comment toggles its display.

eg.

`delete edit reply (n)Show `

Comments

PA robot’s picture

Status: Needs review » Needs work

Link to the project page and git clone command are missing in the issue summary, please add them.

We are currently quite busy with all the project applications and we prefer projects with a review bonus. Please help reviewing and put yourself on the high priority list, then we will take a look at your project right away :-)

Also, you should get your friends, colleagues or other community members involved to review this application. Let them go through the review checklist and post a comment that sets this issue to "needs work" (they found some problems with the project) or "reviewed & tested by the community" (they found no major flaws).

I'm a robot and this is an automated message from Project Applications Scraper.

bogdanru’s picture

Hi,

You should check your module on ventral.org: http://ventral.org/pareview/httpgitdrupalorgsandboxswastikpareek2021269 ... there are some errors there.

First things to do:
- is to change your branch, checkout this link: https://drupal.org/node/1127732
- edit your issue and add link to your project: https://drupal.org/sandbox/swastikpareek/2021269
and git clone command: git clone http://git.drupal.org/sandbox/swastikpareek/2021269.git comment_reply_count

Cheers.

garnett2125’s picture

Hi,

Manual review :

  1. You don't need to put '' in .info file : core = '7.x' => core = 7.x for example
  2. drupal_add_js(drupal_get_path('module','custom_module') . '/js/comment_reply_count.js'); Line 8 comment_reply_count.module, the second argument is the name of the module and you put custom_module, you have to put your module's name
  3. 'reply ('.$result.') Show'; => all user facing text must run through t() for translation
swastik1608’s picture

Thanks for the valuable advice .. I will keep these things in mind while creating other modules

PA robot’s picture

Issue summary: View changes
Status: Needs work » Closed (won't fix)

Closing due to lack of activity. Feel free to reopen if you are still working on this application (see also the project application workflow).

I'm a robot and this is an automated message from Project Applications Scraper.