Git Information
git clone --branch 7.x-1.x https://git.drupal.org/sandbox/puspanjalim/2850107.git fbpagefeed
cd fbpagefeed
This module is a very simple module having small but most useful feature which perfectly works with multiple sites running on same server having common database but different domain name all together.
If you would like to use different Facebook page posts( public ) for different sites then this is the simple and most useful module.
Facebook page feed module allows to show Facebook public page posts in blocks or pages.The most important feature includes assigning different Facebook page url to different domains running on single hosted server.
Project link:
https://www.drupal.org/sandbox/puspanjalim/2850107
Manual reviews of other projects:
https://www.drupal.org/node/2853833#comment-11952543
https://www.drupal.org/node/2855076#comment-11952515
https://www.drupal.org/node/2840555#comment-11952952
https://www.drupal.org/node/2851856#comment-11953419
Comments
Comment #2
puspanjalim CreditAttribution: puspanjalim as a volunteer commentedComment #3
sjpagan CreditAttribution: sjpagan commentedComment #4
sjpagan CreditAttribution: sjpagan commentedfix this errors
using tool https://pareview.sh/
Comment #5
sjpagan CreditAttribution: sjpagan commentedComment #6
sjpagan CreditAttribution: sjpagan commentedComment #7
puspanjalim CreditAttribution: puspanjalim as a volunteer commentedThanks sjpagan.
Comment #8
puspanjalim CreditAttribution: puspanjalim as a volunteer commentedComment #9
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 #10
sjpagan CreditAttribution: sjpagan commentedHi @puspanjalim,
Very well, i see your module's file,
a). I think its a good idea to implement hook_help();
b). In fbpagefeed.info include "configure = admin/config/content/fbpagefeed'.
c). integrate hook_permission();
see you.
Comment #11
puspanjalim CreditAttribution: puspanjalim as a volunteer commentedComment #12
puspanjalim CreditAttribution: puspanjalim as a volunteer commentedComment #13
puspanjalim CreditAttribution: puspanjalim as a volunteer commentedComment #14
puspanjalim CreditAttribution: puspanjalim as a volunteer commentedComment #15
puspanjalim CreditAttribution: puspanjalim as a volunteer commentedComment #16
VoidE CreditAttribution: VoidE commentedGood to see you implemented hook_help. I think it's a good idea to make the content translatable. Just take a look at the documentation for an example.
You also implemented hook_permission and added the permission 'administer fbpagefeed', but in hook_menu, you used the access argument 'administer users'. See:
Please add the t-function to your placeholder attribute in 'fbpagefeed_form' (line 62).
Comment #17
VoidE CreditAttribution: VoidE commentedComment #18
puspanjalim CreditAttribution: puspanjalim as a volunteer commentedThanks. I have made required changes and updated.
Comment #19
puspanjalim CreditAttribution: puspanjalim as a volunteer commentedplease have a look on my module and authorize me to create full project.
Thanks,
Puspanjali
Comment #20
Ankush_03@puspanjalim,
use js file for facebook script, do not write script tag on tpl file.
Comment #21
Ankush_03Comment #22
klausi@agautam: that alone is surely not an application blocker, anything else that you found or should this be RTBC instead?
Comment #23
puspanjalim CreditAttribution: puspanjalim as a volunteer commentedComment #24
rajveergangwar@Puspanjalim,
Below are my reviews:
1) Please use t() in hook_help.
tag in the tpl file , which you should not used, in tpl file. you should add it from drupal_add_js() as inline. 4) in module.info file description should be more descriptive , as it is not cleared module works for multi domain too.2) variable which is set by module , must be deleted after module unistall , use hook_uninstall in module.install file.
3) You have used
Comment #25
tatarbjhelping to set the right status to this issue as it waits for the maintainer response.
Comment #26
PA robot CreditAttribution: PA robot commentedClosing due to lack of activity. If you are still working on this application, you should fix all known problems and then set the status to "Needs review". (See also the project application workflow).
I'm a robot and this is an automated message from Project Applications Scraper.