1. Extends the existing ShareThis module. Adds ShareThis as a field type, when creating content types. Allow a node author the ability to choose whether or not to show ShareThis on an individual node basis.

2. N/A

3. Project page: http://drupal.org/sandbox/kjmeath/1938534

4. Git repo: git.drupal.org:sandbox/kjmeath/1938534.git

5. Drupal 7

6. N/A

Comments

likebtn’s picture

likebtn commented

It would be good to have a screenshot or link to the Live Demo on the Project page: http://drupal.org/sandbox/kjmeath/1938534

Results of the automated review: http://git.drupal.org/sandbox/kjmeath/1938534.git
Take a look at them and make sure to fix errors.

README.txt is missing, see the guidelines for in-project documentation.

Module depends on sharethis module, add this info to the Project page and README.txt

Manual review:
1) sharethis_field.module
- Consider justifying tabs and indents of the arrays (Line 11 and so on)
2) I've installed module, added ShareThis field to the Basic page and to the Article Content Type, but share buttons by some reason are not displayed for them.

arnoldbird’s picture

arnoldbird commented
Status: Needs review » Needs work

Here's a working link to an automated review:
http://ventral.org/pareview/httpgitdrupalorgsandboxkjmeath1938534git

Based on that review and likebtn's review, I'm changing this to 'needs work.'

Anonymous’s picture

Anonymous (not verified) commented

Thanks, I'll get these issues fixed up.

PA robot’s picture

PA robot commented
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.

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