Social Feed Block module provides the user to fetch the data from their respective Facebook, Twitter, Pintrest and Instagram profiles and then display them accordingly as per their requirement using the Drupal block system .

Project Page

https://www.drupal.org/sandbox/jsvr22/2742579

pareview.sh review

https://pareview.sh/node/922

Git Instructions

git clone --branch 8.x-1.x https://git.drupal.org/sandbox/jsvr22/2742579.git social_feeds_block
cd social_feeds_block

How to Use

Here are the steps to use Social Feed Block module.
Install:
1. Clone/Download module to /modules directory
2. Enable Social Feed Block module (via drush or by Drupal's Extend page)

Configure:
1. Set all the credentials ( admin/config/system/fb-social-feeds-api-cridentials )

Test:
1. Goto Administration > Structure > Block
2. Place the block ( named : Facebook , Instagram ,Twitter or Pintrest ).

Comments

jitendra verma created an issue. See original summary.

jitendra verma’s picture

jitendra verma commented
Issue summary: View changes
jitendra verma’s picture

jitendra verma commented
Issue summary: View changes
jitendra verma’s picture

jitendra verma commented
Issue summary: View changes
jitendra verma’s picture

jitendra verma commented
Project: Social Feeds Block » Drupal.org security advisory coverage applications
Component: Code » module
PA robot’s picture

PA robot commented
Issue summary: View changes
Status: Needs review » Needs work

There are some errors reported by automated review tools, did you already check them? See http://pareview.sh/pareview/httpsgitdrupalorgsandboxjsvr222742579git

Fixed the git clone URL in the issue summary for non-maintainer users.

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.

andrew_tspkh’s picture

andrew_tspkh
Primary language Ukrainian
commented

Hi, @jitendra verma
Please, use dependency injection instead of \Drupal calls according to Drupal Best Practices (FacebookFeedsBlock.php [line:21, line:30], InstagramFeedsBlock.php [line:21, line:32, line:35], PintrestFeedsBlock.php [line:21, line:26], TwitterFeedsBlock.php [line:21])
Remove all commented code in the files.
Remove unnecessary # in social_feeds_block.routing.yml file.
Thanks.

b n pandey’s picture

b n pandey commented

HI Jitendra,

I have reviewed your module manually and found some issues.

1: Please describe your variable in twig file inside comment section.
2: I found some Bad line endings were found issue in https://pareview.sh.
3: You have used services directly on some places because you used "\Drupal::config('config.twitter_social_feeds');" inside class.

Thanks
BN Pandey

PA robot’s picture

PA robot commented
Status: Needs work » Closed (won't fix)

Closing 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.