Closed (duplicate)
Project:
Drupal.org security advisory coverage applications
Component:
theme
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
26 Jun 2013 at 16:18 UTC
Updated:
9 Sep 2018 at 23:25 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
PA robot commentedThere are some errors reported by automated review tools, did you already check them? See http://ventral.org/pareview/httpgitdrupalorgsandboxrameshrasaiyan2023639git
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
rameshrasaiyan commentedCorrected the git repository and added the theme name.
Comment #2.0
rameshrasaiyan commentedUpdated the git repository and added the theme name.
Comment #3
Cristobal Wetzig commentedHi!
Browsed trough your theme, Some things i found
Nice touch with the breadcrumb changer in the settings!
Comment #4
Cristobal Wetzig commentedchanged status
Comment #5
rameshrasaiyan commentedHi Cristobal Wetzig,
Thank you for reviewing my theme. I have fixed all your points and updated the repository.
Thanks,
Ramesh
Comment #6
itapplication commentedHi Ramesh ,
I downloaded your theme and tested it manually on my localhost XAMPP server. It's good looking theme I relay appreciate it. Nice work. I didn't find any major flaw/defect but need some usability improvement into theme like -
Except above minor things theme is excellent!
itapplication
(Shiv)
Comment #7
rameshrasaiyan commentedHi Shiv,
Thank you for your valuable review comments. I have fixed all your points and updated the repository with the latest code.
Please review it and let me know if anything missed out.
Thanks,
Ramesh
Comment #8
itapplication commentedHi Ramesh,
Thanks for your tireless efforts to fixing bugs. Theme is really better now, I appreciate your work.
I test again download/pull theme repository and test manually on my XAMPP server. Every thing looks good except one minor thing. Highlighted region should be style to stand out it from other area of them. We called it highlighted so make styling to highlight it.
Except above minor bug theme is excellent.
Thanks.
Shiv
Comment #9
rameshrasaiyan commentedHi Shiv,
Thank you for the quick review. I have fixed the Highlighted region skinning and updated the repository.
Please review it and let me know if anything missed out.
Thanks,
Ramesh
Comment #10
webengage_nitin commentedHi,
Had a glance at your module. Really liked it. Just make sure you correct all the errors reported by automated review tools. There are still many, majorily corresponding to line formatting: automated review
Regards,
Nitin
Comment #11
rameshrasaiyan commentedHi Nitin,
Thanks for reviewing my theme.
All the automated review comments has been fixed and it is ready for review. Please review it and let me know if anything needs be done.
Thanks,
Ramesh
Comment #12
PA robot commentedProject 1: https://drupal.org/node/2072859
Project 2: https://drupal.org/node/2028851
As successful completion of the project application process results in the applicant being granted the 'Create Full Projects' permission, there is no need to take multiple applications through the process. Once the first application has been successfully approved, then the applicant can promote other projects without review. Because of this, posting multiple applications is not necessary, and results in additional workload for reviewers ... which in turn results in longer wait times for everyone in the queue. With this in mind, your secondary applications have been marked as 'closed(duplicate)', with only one application left open (chosen at random).
If you prefer that we proceed through this review process with a different application than the one which was left open, then feel free to close the 'open' application as a duplicate, and re-open one of the project applications which had been closed.
I'm a robot and this is an automated message from Project Applications Scraper.
Comment #12.0
PA robot commentedCorrected the git repository
Comment #13
avpaderno