Skip to content

Fix Instagram widget not updating (v2)#246

Merged
violetshine merged 2 commits intomasterfrom
fix-instagram-widget-v2
Apr 9, 2026
Merged

Fix Instagram widget not updating (v2)#246
violetshine merged 2 commits intomasterfrom
fix-instagram-widget-v2

Conversation

@violetshine
Copy link
Copy Markdown
Contributor

Remove SnapWidget since it seems to have become paid and stopped pulling updated from Instagram a long long time ago. Use Instagram's first-party embedding feature instead of a third-party service.

Redid this PR because of the Astro migration forcing me to delete my local clone of the repo and start all over again.

Fixes: #213

@violetshine violetshine requested a review from akselotti April 7, 2026 23:22
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 7, 2026

Azure Static Web Apps: Your stage site is ready! Visit it here: https://green-sea-053340d03-246.westeurope.1.azurestaticapps.net

Copy link
Copy Markdown
Contributor

@akselotti akselotti 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 but could you solve the npm run check errors

Remove SnapWidget since it seems to have become paid and stopped
pulling updated from Instagram a long long time ago.

Use Instagram's first-party embedding feature instead of a
third-party service.

Fixes: #213
@violetshine violetshine force-pushed the fix-instagram-widget-v2 branch from 1d37db1 to 92cc02e Compare April 8, 2026 07:40
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 8, 2026

Azure Static Web Apps: Your stage site is ready! Visit it here: https://green-sea-053340d03-246.westeurope.1.azurestaticapps.net

@violetshine
Copy link
Copy Markdown
Contributor Author

Oops, Astro does the importing a little bit differently than the old system. Fixed that and amended my commit.

It still does hint about the iframe being scrolling="no" if running npm run check locally because according to some HTML5 spec that should be done with CSS instead but it seems like Chrome still has not yet implemented the same in CSS 13 years later and expects it to be scrolling="no" even thoughscrolling is meant to be deprecated. Can't do anything about that, it's Google's problem to deal with.

@violetshine violetshine requested a review from akselotti April 8, 2026 08:33
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 8, 2026

Azure Static Web Apps: Your stage site is ready! Visit it here: https://green-sea-053340d03-246.westeurope.1.azurestaticapps.net

Copy link
Copy Markdown
Contributor

@akselotti akselotti left a comment

Choose a reason for hiding this comment

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

lgtm

@violetshine violetshine merged commit 552d6d1 into master Apr 9, 2026
8 checks passed
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.

Instagram tiles not updating

3 participants