Skip to content

Stabilize the news page by removing the LinkedIn widget dependency#178

Draft
Copilot wants to merge 2 commits intomainfrom
copilot/fix-linkedin-news-rendering
Draft

Stabilize the news page by removing the LinkedIn widget dependency#178
Copilot wants to merge 2 commits intomainfrom
copilot/fix-linkedin-news-rendering

Conversation

Copy link
Contributor

Copilot AI commented Mar 9, 2026

PR Summary

The news page was depending on an external LinkedIn widget that no longer rendered reliably, leaving the page stale and visually broken. This change makes the site-owned Jekyll news archive the primary source of truth and documents the maintenance path for future updates.

  • Motivation:

    • Eliminate the broken/outdated LinkedIn embed from 3-news.html
    • Keep the news page current using content that lives in the repo and deploys with the site
  • Main changes:

    • News page content
      • Remove the third-party SociableKit LinkedIn embed and its fallback script from 3-news.md
      • Replace it with a static intro, a direct LinkedIn follow link, and guidance that the cards below are the canonical archive
    • Maintenance guidance
      • Add README.md guidance for publishing news through _posts/ with WebP assets
      • Make the intended workflow explicit so routine updates do not depend on external widgets
    • Example
      <ul class="actions">
        <li><a href="{{ site.socials.LinkedIn }}" class="button next" target="_blank" rel="noopener noreferrer">Follow SECQUOIA on LinkedIn</a></li>
      </ul>
  • Related issues (if any):

    • Addresses the broken/missing LinkedIn news rendering on 3-news.html

Preview Artifact

This PR triggers the Preview Build workflow.

  • Download the preview-site artifact from the workflow run to inspect the generated site.

  • To view locally: Extract the artifact and serve it with a local web server:

    # Extract the zip file, then run one of:
    python3 -m http.server 8000
    # or
    npx serve
    # or
    php -S localhost:8000

    Then browse to http://localhost:8000

  • Note: Opening HTML files directly (file://) won't work correctly due to absolute paths.

  • Internal and external link check logs are in the preview-link-check-logs artifact.

  • Link checks are non-blocking; fix issues where practical before merge.

Image & Asset Guidelines

Only WebP images should be referenced in the site; originals (PNG/JPG/etc.) may be stored for provenance.

  1. Always reference the .webp version in Markdown/HTML.

  2. Originals can live alongside WebP conversions but must NOT be used directly in content.

  3. Prefer descriptive, kebab-case filenames (e.g., joao-victor-paim.webp).

  4. If adding a new image, convert to WebP first. Example conversion commands:

    # Using cwebp
    cwebp input.jpg -q 80 -o output.webp
    
    # Using ImageMagick
    magick input.png -quality 80 output.webp
  5. Run the existing resize helper if appropriate:

    ./resize_images.sh

    (Ensure the script suits the new image before relying on it.)

Checks Before Requesting Review

  • Jekyll preview build succeeds locally (bundle exec jekyll build).
  • All referenced images are WebP (originals not directly referenced).
  • All member/profile links verified.
  • External links added are valid (ignore transient 429/timeout warnings unless persistent).
  • No large binary files (>5MB) added without justification.

Follow-up / Deployment Notes

  • Future news updates should be added as dated Jekyll posts in _posts/; LinkedIn can remain a distribution channel, but not the rendering source for the page.

Screenshots (If UI/Content Changes)


Maintainers: Ensure branch protection requires only CI + Preview jobs, not production Pages deployment for PRs.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • secquoia.github.io
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node /home/REDACTED/work/_temp/ghcca-node/node/bin/node --enable-source-maps /home/REDACTED/work/_temp/copilot-developer-action-main/dist/index.js (dns block)
  • widgets.sociablekit.com
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node /home/REDACTED/work/_temp/ghcca-node/node/bin/node --enable-source-maps /home/REDACTED/work/_temp/copilot-developer-action-main/dist/index.js (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

This section details on the original issue you should resolve

<issue_title>[BUG]</issue_title>
<issue_description>Describe the bug
The updated linkedin news are not rendering well (logo missing, out dated).

To Reproduce
Steps to reproduce the behavior:

  1. Go to https://secquoia.github.io/3-news.html

Expected behavior
We would like to have this page updated and rendering well. Either have it in the instructions for usual maintanance (in the README), try to update it regularly through GitHub CI, or find a better alternative
</issue_description>

Comments on the Issue (you are @copilot in this section)


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Co-authored-by: bernalde <8647329+bernalde@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix rendering issues with updated LinkedIn news Stabilize the news page by removing the LinkedIn widget dependency Mar 9, 2026
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.

[BUG]

2 participants