Closed (won't fix)
Project:
Drupal.org security advisory coverage applications
Component:
module
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
9 Jun 2011 at 12:23 UTC
Updated:
10 Sep 2018 at 10:06 UTC
Jump to comment: Most recent
Comments
Comment #1
ccardea commentedThank you for your application. The wait time for a review is now approaching seven weeks. You can help your own cause by contributing to the code review process. All it takes is basic module writing skills. Please visit http://groups.drupal.org/code-review for details on how to participate.
Comment #2
rightchoice2c_me commentedthanks for your comment. Sure I'll look into that. Expecting my module to be reviewed earlier than seven weeks... :-)
Comment #3
jordojuice commentedUpdating priority according to priority guidelines. The application's priority should be set back to normal once a reviewer responds to your application and the application review process has continued.
Comment #4
ParisLiakos commentedComment #5
davidhernandezLooking at this now; thanks for being patient. Did you write the plugin yourself? I was surprised to see this functionality didn't already exist.
Comment #6
rightchoice2c_me commentedThanks for your reply...
Yes.
Comment #7
klausi* git release branch missing, see http://drupal.org/node/1015226
* plugin.js: indentation errors, should be 2 spaces, see javascript coding standards http://drupal.org/node/172169
* module: "if (($editor == 'ckeditor'))" why the double "()"?
Comment #8
misc commentedrightchoice2c_me has been contacted to ask if the application is abandoned.
http://drupal.org/node/894256
Comment #9
rightchoice2c_me commentedThank you very much MiSc for notifying me ...
I'm very much busy these days, Hopefully, will fix the items mentioned by klausi in a day or two and update the git and here as well...
Comment #10
rightchoice2c_me commentedI'm actually waiting for client approval on this module, they are verifying the code and once they give green signal, I'll update here.... Thanks
Comment #11
davidhernandezIf you are waiting on something you can change the status to "postponed". When you are ready to get reviewed again you can switch back to "needs review".
Comment #12
rightchoice2c_me commentedchanging the status to postponed, as waiting for client approval.
Thanks
Comment #13
klausiClosing due to lack of activity. Feel free to reopen if you are still working on this application.
Comment #14
avpaderno