Instagram Lite

Almost all Instagram feed plugins that do not require the Instagram API have been broken since the latest Instagram changes.

The already existing instagram feed plugins that were worked without API got broken since the latest Instagram changes. To address this issue you can use this module to display your Instagram feed using the new Instagram Graph API.

Similar projects and how they are different

  1. Simple Instagram Feed Block: Using jQuery plugin to fetch feed from Instagram and it is broken now due to new Instagram API changes.
  2. Instagram API: Based on old instagram API

Project link

https://www.drupal.org/project/instagram_lite

Git instructions

git clone --branch '1.0.x' https://git.drupalcode.org/project/instagram_lite.git

PAReview checklist

http://pareview.net/r/215

Comments

keen-on.digital created an issue. See original summary.

keen-on.digital’s picture

Status: Active » Needs review
avpaderno’s picture

Thank you for applying! Is keen-on.digital a shared account?

keen-on.digital’s picture

@kiamlaluno, Thanks for your comment. No, keen-on.digital is not a shared account.

rajeshreeputra’s picture

PAreview checklist link, showing no issues.

Thanks,
Rajeshreeputra

avpaderno’s picture

Priority: Normal » Critical

I changed the issue priority as described on Review process for security advisory coverage: What to expect / Application Review Timelines.

To reviewers: Please change back the priority to Normal after reviewing the project.

avpaderno’s picture

Priority: Critical » Normal
Status: Needs review » Needs work
  • What follows is a quick review of the project; it doesn't mean to be complete
  • For every point, the review doesn't make a complete list of lines that should be fixed, but an example of what is wrong in the code
  • A review is about code that doesn't follow the coding standards, contains possible security issue, or doesn't correctly use the Drupal API; if a point isn't about that, it makes it clear
  public function testLoadFrontPage() {
    $this->drupalGet(Url::fromRoute('<front>'));
    // Confirm that the page loaded correctly.
    $this->assertSession()->statusCodeEquals(200);
    // Confirm that the front page contains the standard text.
    $this->assertSession()->pageTextContains('Welcome to Drupal');
  }

That test isn't testing the module, since the module doesn't change the front page. Drupal core already tests the front page in FrontPageTest.
The module should test if its block is rendered correctly, similarly to what HelpBlockTest does.

keen-on.digital’s picture

Status: Needs work » Needs review

@apaderno, thanks for your comment.

Test has been updated to test the site along with instagram_lite block.

avpaderno’s picture

Assigned: Unassigned » avpaderno
Status: Needs review » Fixed

Thank you for your contribution! I am going to update your account.

These are some recommended readings to help with excellent maintainership:

You can find more contributors chatting on the IRC #drupal-contribute channel. So, come hang out and stay involved.
Thank you, 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.

I thank all the dedicated reviewers as well.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.