Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
YouTube channel subscribe button using token. You can place token [youtube:button] anywhere in body.
You can easily place YouTube subscribe button without any code. Just need to add token to your body and also set channel name or channel id in admin configuration page.
For clone this project use following:
git clone http://git.drupal.org/sandbox/ajay.gadhavana/2550017.git
sandbox link :
https://www.drupal.org/sandbox/ajay.gadhavana/2550017
Test of pareview.sh:
http://pareview.sh/pareview/httpgitdrupalorgsandboxajaygadhavana2550017git
Comment | File | Size | Author |
---|---|---|---|
#3 | empty-repository.png | 12.65 KB | ARUN AK |
Comments
Comment #2
Ajay Gadhavana CreditAttribution: Ajay Gadhavana commentedComment #3
ARUN AK CreditAttribution: ARUN AK commentedIt seems like your GIT repository is empty. Please upload your code in to the git repository, then only can proceed with further review.
Thanks,
ARUN AK
Comment #4
ajalan065 CreditAttribution: ajalan065 as a volunteer and at Innoraft commentedHi Ajay Gadhavana,
Your project page is missing the most crucial part of any project, the git clone link. On visiting the Version Control Tab of your module, I found it missing there too. Please follow proper instructions for the same.
However, I downloaded the zip file of your project, and found quite errors.
1. Remove the README.txt~ from your project.
2. Remove the version tag from the *.info file.
3. Comments missing in the *.module and *.inc file.
4. White spaces missing here and there. Please perform Pareview Test to know the details.
5. In the *.inc file, please use get_t() with the description tag.
6. Use isset() instead of array_key_exists()/ empty() in *.module file.
Comment #5
irfworld CreditAttribution: irfworld commentedHi Ajay,
Kindly add the git clone link as well as pareview.sh link too.
Comment #6
Ajay Gadhavana CreditAttribution: Ajay Gadhavana commented1) I've solved all queries of ajalan065 but still cant figure out Point 4.
2) Added module to git repository.
3) irfworld where should I add the git clone link as well as pareview.sh link too.
Comment #7
irfworld CreditAttribution: irfworld commented@Ajay,
You have to add the links in your project description section.
Or you can read the instructions on below links:
https://www.drupal.org/node/632262
Comment #8
Ajay Gadhavana CreditAttribution: Ajay Gadhavana commentedComment #9
Ajay Gadhavana CreditAttribution: Ajay Gadhavana commentedRequested changes are done. Please check the same.
Comment #10
Ajay Gadhavana CreditAttribution: Ajay Gadhavana commentedComment #11
PA robot CreditAttribution: 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 #12
sheldonkreger CreditAttribution: sheldonkreger as a volunteer commentedSandbox git link is broken.
http://git.drupal.org/sandbox/ajay.gadhavana/2550017.git
We also need a link to your sandbox project.
Comment #13
Ajay Gadhavana CreditAttribution: Ajay Gadhavana commentedComment #14
Ajay Gadhavana CreditAttribution: Ajay Gadhavana commentedSandbox link also added please check the same
Comment #15
ashishdalviHi,
Functionality looks very good. Please find my comments on module review.
Automated Review
No issues found in automation review
http://pareview.sh/pareview/httpgitdrupalorgsandboxajaygadhavana2550017git
Manual Review
1. Incorrect access argument used in youtube_channel_subscribe_menu()
'access arguments' => array('Administration Menu'),
1. Naming conventation for inc file name should be youtube_channel_subscribe.admin.inc
'file' => 'youtube_channel_subscribe_admin.inc',
This review uses the Project Application Review Template.
Thanks,
Ashish
Comment #16
Ajay Gadhavana CreditAttribution: Ajay Gadhavana commentedThanks Ashish,
I've solved errors that's defined in Coding style & Drupal API usage.
Thanks for your help.
Comment #17
Ajay Gadhavana CreditAttribution: Ajay Gadhavana commentedHello Team,
I have done all the things those are listed,
Is there anything remaining?
Thanks,
Ajay
Comment #18
Ajay Gadhavana CreditAttribution: Ajay Gadhavana commentedComment #19
sysosmaster CreditAttribution: sysosmaster commentedManual Review
The starred items (*) are fairly big issues and warrant going back to Needs Work. Items marked with a plus sign (+) are important and should be addressed before a stable project release. The rest of the comments in the code walkthrough are recommendations.
If added, please don't remove the security tag, we keep that for statistics and to show examples of security problems.
This review uses the Project Application Review Template.
Comment #20
sysosmaster CreditAttribution: sysosmaster commentedComment #21
DamienMcKennaThanks for your contribution, Ajay!
I updated your account so you can promote this to a full project and also create new projects as either a sandbox or a "full" project.
Here are some recommended readings to help with excellent maintainership:
You can find lots more contributors chatting on IRC in #drupal-contribute. So, come hang out and stay involved!
Thanks, also, for your patience with the review process. Anyone is welcome to participate in the review process. Please consider reviewing other projects that are pending review. I encourage you to learn more about that process and join the group of reviewers.
Thanks to the dedicated reviewer(s) as well.
Comment #22
DamienMcKennaComment #24
Ajay Gadhavana CreditAttribution: Ajay Gadhavana commentedCould someone let me know How I can submit same module as D8 version, need to follow full process same as D7 module or just need to create branch ?
Please suggest me. I have already converted module to d8 version but not sure how to upgrade here.
Thanks in advance..
Comment #25
DamienMcKenna@Ajay: Just create a new branch, you don't have to go through the process again.