Skip to content

Feature/Instagram-widget#67

Open
GladwynChua wants to merge 5 commits into
mainfrom
feature/instagram-widget
Open

Feature/Instagram-widget#67
GladwynChua wants to merge 5 commits into
mainfrom
feature/instagram-widget

Conversation

@GladwynChua
Copy link
Copy Markdown
Contributor

@GladwynChua GladwynChua commented May 12, 2026

What does this PR do?

Implemented the instagram widget (elfsight) onto the home page. Currently has sliding effect, open to suggestions. '

The two instagram widgets I came across were sociablekit and elfisght which were both free options. Both widgets could generate grid formats without having to authenticate and sign in to SSA Instagram. I ultimately decided to use Elfsight as Sociablekit were causing issues where the widget would fail to load images when navigating back to the home page. The widget script was being injected via useEffect, however the widget itself didn't re-initialize properly. Also, free tier also had issues around refresh rate.

Elfsight was loads reliably without manual reinitialisation. It has a dashboard which can update layout without touching code. Easier to implement code without the requirement of useEffect.

Type of change

  • Feature
  • Bug fix
  • Chore / config
  • Hotfix

Checklist

  • My branch follows the naming convention (feature/, fix/, chore/, hotfix/)
  • My commit messages follow the conventional commits format
  • CI checks pass

Linked Issues

Closes #56

@GladwynChua GladwynChua changed the title feat(Instagram-widget): implemented Instagram widget using elfsight Feature/Instagram-widget May 12, 2026
oorjagandhi
oorjagandhi previously approved these changes May 12, 2026
Copy link
Copy Markdown
Collaborator

@oorjagandhi oorjagandhi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks awesome nice work!

Just a couple of small non-technical requests before merging:

  • Document your decision - could you add a brief explanation to the PR description explaining why you chose Elfsight? Since we don't have formal docs, the PR description is a good place to capture this so future developers understand the reasoning.
  • Link your ticket in the PR description

Good stuff Gladwyn

oorjagandhi
oorjagandhi previously approved these changes May 13, 2026
Copy link
Copy Markdown
Collaborator

@oorjagandhi oorjagandhi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me! Nice work on this Gladwyn

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

HOME-02 - Instagram Widget

2 participants