Name : NN Payment Module - PCI

NN Payment Module - PCI is an drupalcommerce payment module which supports redirect type of Payment Processing.

Sandbox Link :http://drupalcode.org/sandbox/arun_me/1492560.git/commit/4598ed6

Git clone Link :git clone --branch 7.x-1.x arun_me@git.drupal.org:sandbox/arun_me/1492560.git nn_payment_module_pci

Git Link : http://git.drupal.org/sandbox/arun_me/1492560.git

CommentFileSizeAuthor
#5 parareview.txt234.23 KBluxpaparazzi

Comments

nmudgal’s picture

Assigned: arun_me » Unassigned
Status: Needs review » Needs work

Welcome to community :-)
You didn't give any project page link, so have search it, anyways.
http://drupal.org/sandbox/arun_me/1492560

Make sure you follow instructions given here http://ventral.org/pareview/httpgitdrupalorgsandboxarunme1492560git & resolve all these issues until someone do in-depth review of your module.

Also, please don't assign issue to yourself.

Thanks

arun_me’s picture

Status: Needs work » Needs review

Hello All,

We cleared al the errors shown by PAREVIEW.So please review this application and post us the status.

Thanks

arun_me’s picture

Hello nmudgal,

We are waiting for the Review result.We cleared all errors thrown by PAREVIEW. But long Time we didnt get any result from your side.So please guide us as soon as possible.

Thank you,

patrickd’s picture

Hi,
I'm very sorry for the delay, there are currently hundreds of applications in the queue, so it's gonna take some time.
We do really need more hands in the application queue and highly recommend to get a review bonus so we will(/can) come back to your application sooner.
regards

luxpaparazzi’s picture

Status: Needs review » Needs work
StatusFileSize
new234.23 KB

1. There are many coding standard errors, as detected by paraview, see attached file!
2. You could spend a word explaining what "Novalnet Credit Card Pci" actually is, see also:

project page
Please take a moment to make your project page follow tips for a great project page.

3. Your project page link is not working!
4. You should not include GPL licence in readme.txt

luxpaparazzi’s picture

Please also put the ":" where they belong to in your project application!!

"Released under the GNU General Public License" => remove this, as all projects on drupal.org must be GPL

Recheck the layout of your README.txt, looks badly formatted!

The response time for a review is now approaching 4 weeks.
Get a review bonus and we will come back to your application sooner.
See: http://drupal.org/node/1410826

You could for example start by evaluating my own project:
http://drupal.org/node/1302786

arun_me’s picture

Hello all,

We updated the changes, Please review our application.

Thanks,

klausi’s picture

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.

avpaderno’s picture

Title: NN Payment Module - PCI » [D7] NN Payment Module - PCI
Issue summary: View changes