Skip to content

fix(site): restore logos and clean event data#174

Merged
sergiocarracedo merged 5 commits intoVigoTech:mainfrom
sergiocarracedo:fix/fixes-and-old-pending-prs
Apr 9, 2026
Merged

fix(site): restore logos and clean event data#174
sergiocarracedo merged 5 commits intoVigoTech:mainfrom
sergiocarracedo:fix/fixes-and-old-pending-prs

Conversation

@sergiocarracedo
Copy link
Copy Markdown
Collaborator

Summary

This PR focuses on fixing broken group logo rendering and cleaning up generated event data so duplicated or stale event entries no longer appear.
It also includes the related branch updates already present around Meetup event identity handling, generated content refreshes, and GitHub Actions workflow modernization.

Changes

Fixes

  • Update VigoTech-hosted group logo URLs to the new /images/groups/... path convention
  • Ensure logo normalization handles both legacy /images/... URLs and the new /images/groups/... URLs
  • Keep group logo avatars square and contained so logos remain visible instead of being cropped or distorted
  • Fix duplicated generated events by deduping stable event identities and pruning stale generated markdown files
  • Restore missing Meetup-backed event entries after the metagroup-schema-tools source ID update

Chores

  • Refresh generated event and video content after the event identity and logo-path changes
  • Keep the branch’s GitHub Actions upgrades and pnpm cache setup fixes already included here

PR Details (AI generated)

Goals

This PR fixes two user-visible issues on the site. First, some community logos, especially VigoWordPress in event cards, were not rendering correctly because the project was still carrying a mix of legacy logo URLs and newer local asset paths. Second, generated event content had started to accumulate duplicate or stale entries after upstream event identifiers changed, which caused duplicated event cards and inconsistent generated markdown files.

Changes in detail

Fixes

  • Updated public/vigotech.json so VigoTech-hosted group logos now point to the new https://vigotech.org/images/groups/... path layout
  • Extended toLocalGroupLogo(...) in both the runtime source helper and the generator so it understands legacy URLs, new group URLs, and already-local /images/groups/... paths
  • Updated GroupLogo.astro so it normalizes incoming logo paths before rendering and keeps the avatar frame square with contained imagery
  • Improved generated-event deduplication in scripts/generate-vigotech-json.mjs so events are merged by stable identity using link first and normalized title plus date as fallback
  • Added cleanup for stale generated markdown files so old event files are removed when canonical generated identifiers change
  • Refreshed generated event data so duplicated A Industriosa and VigoWordPress event entries are removed from the generated output

Chores

  • Includes the already-present branch updates around metagroup-schema-tools@1.1.1, generated content refreshes, and workflow action upgrades
  • Keeps the Pages workflow updates to newer action versions and the pnpm setup ordering fix on the same branch

Notes

This branch already contains several related fixes that were built incrementally, so the cleanest review path is to focus on the latest user-visible outcomes: logo rendering, square/contained avatars, and duplicate event cleanup. Generated content changes are expected because the fixes affect both logo source paths and event identity handling.

@sergiocarracedo sergiocarracedo merged commit a75eb74 into VigoTech:main Apr 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.

1 participant