Closed (won't fix)
Project:
Drupal.org security advisory coverage applications
Component:
module
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
10 Feb 2014 at 14:45 UTC
Updated:
27 May 2014 at 16:27 UTC
Jump to comment: Most recent
Comments
Comment #1
PA robot commentedThere are some errors reported by automated review tools, did you already check them? See http://pareview.sh/pareview/httpgitdrupalorgsandboxcaptcha-in-thai219270...
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.
Comment #2
captcha_in_thai commentedI fixed pareview recommendation for all files
Comment #3
auworks commentedHi mate,
Just did a install and code review of you module and found you have used thai language directly in your module code...
I believe this should be handled using .po files, please refer to this article: http://stackoverflow.com/questions/5231496/how-to-generate-translation-f...
@klausi can you please confirm....
Is there any reason why this plugin was not developed as a submodule for CAPTCHA module?
Cheers,
Ash
Comment #4
auworks commentedComment #5
auworks commentedComment #6
PA robot commentedClosing 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.