HHS Digital Media Syndication is a module for displaying content from the U.S. Department of Health and Human Services. The module allows you to search and find html snippets, images, infographics, buttons, badges, ecards, and other media types. The content displayed will be dynamically retrieved from the source when it is displayed so you always have the most recent content on your site. This is especially important to state and local health agencies that need accurate and up to date content on their sites, but don't have the resources to maintain the content. If a local agency is using drupal for their site they can install this module and have instant access to important health marketing content provided by federal agencies like HHS, CDC, FDA, etc.
The project page is here: https://www.drupal.org/sandbox/bhwilliamson/2332317
Code: git clone --branch 7.x-1.x http://git.drupal.org/sandbox/bhwilliamson/2332317.git
On a side note, this is my first attempt at any drupal development. I've tried to follow all standards and best practices but I'm sure there is room for improvement, so any constructive feedback is greatly appreciated.
Comments
Comment #1
PA robot commentedWe 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.
Comment #2
k_zoltan commentedI looks to me like the author has abandoned this module.
No more commits to the module.
No activity in the issue queue, or on the drupal.org site.
Comment #3
klausiLow activity is not an application blocker, please do a real manual review.
Comment #4
arijits.drushManual Review
Individual user account
Yes: Follows
No duplication
Yes: Does not cause
Master Branch
Yes: Follows
Licensing
Yes: Follows
3rd party assets/code
Yes: Follows
README.txt/README.md
Your readme.txt describe everything correctly but if you can use Readme template that would be even better.
Code long/complex enough for review
Yes: Follows
Secure code
Yes: Meets the security requirements.
Coding style & Drupal API usage
just a recommendation: @file block missing for hhs_digital_media_syndication.js
just a recommendation: to be more nitpicker If you can improve your hook_help() that would be good.
This review uses the Project Application Review Template.
Comment #5
arijits.drushComment #6
bhwilliamson commentedThanks for taking the time to review, I appreciate it and will take a look at implementing your recommendations in a bit. Sorry for the delay in commenting here, for some reason I'm not getting email notifications on issue comments/updates, I need to look into settings to figure out what's going on. I've been working on/off since I submitted this on a 2.x release of this (and about 1000 other projects) and will push those changes up in a bit. I wanted to wait for this initial review to be complete so I'd know if I was on the right track with this. It sounds like now I'll just wait for a git admin to validate the review?
Comment #7
arijits.drushThank you bhwilliamson,
Yes you can wait for any one's attention, or you could do manual review like I did for 3 projects using
Project Application Review Template.
And go to high priority list where drupal.org admin can check.
Although it is not necessary.
But still for faster processing.
As we all part of open source community, other of our community needs attention too.
Thanks
Arijit
Comment #8
cweagansPlease move the contents of the hhs_digital_media_syndication directory to the root of the Git repo. When this is done, use my contact form to let me know and I will approve this application.
Comment #9
PA robot commentedClosing due to lack of activity. If you are still working on this application, you should fix all known problems and then set the status to "Needs review". (See also the project application workflow).
I'm a robot and this is an automated message from Project Applications Scraper.