Skip to content

Review: Sip 'n' Serve Society — Re-check Pass (Step 10)

Reviewer: reviewer agent
Date: 2026-05-25
Pages audited: index.html, membership.html, store.html, calendar.html
Prior review issues: 6 flagged; re-checking all 6 below.

Issue 1: calendar.html has full nav, content, and footer (was: skeleton)

PASS

  • File is 546 lines (was a bare <body> shell before).
  • grep '<nav\|<section\|<footer' returns 8 matches: nav, multiple sections, footer all present.
  • Footer open/close tags confirmed (2 matches).

Issue 2: All CDN image URLs replaced with local paths (assets/images/)

FAIL — calendar.html still has 3 Squarespace CDN refs

Page squarespace refs local assets/images/ refs
index.html 0 16
membership.html 0 3
store.html 0 3
calendar.html 3 2

Remaining CDN refs in calendar.html: - static1.squarespace.com/...sip_n_serve_MainLogo_White_Transparent.png (logo) - images.squarespace-cdn.com/.../nov6-72.jpg (event image) - images.squarespace-cdn.com/.../sep+13-20.jpg (event image)

Issue 3: Nav labels consistent across all pages (was: "Calendar" vs "Schedule" drift)

PARTIAL FAIL — index.html still says "Calendar" while all other pages say "Schedule"

Page Calendar link label
index.html Calendar
membership.html Schedule
store.html Schedule
calendar.html Schedule

index.html desktop nav and mobile overlay both use "Calendar". The other three pages all say "Schedule". The destination (calendar.html) is the same; the label is not.

PASS

All four pages confirmed: Instagram, TikTok, Facebook all present in footer with correct hrefs and aria-labels.

Issue 5: OG/Twitter meta tags present on membership and store

PASS

All four pages return 3 matches for og:title, og:description, twitter:card. Previously only index.html had them.

Issue 6: Form accessibility labels present

PASS (with caveat)

index.html, membership.html, store.html all have 3 <label> elements with class="sr-only" and for attributes matching input IDs (first_name, last_name, email).

calendar.html has 0 <label> elements. If calendar.html has a newsletter form section, it needs labels. If it has no form, not a blocker. Codebuilder should confirm.

Summary

Issue Status
1. calendar.html full content PASS
2. Local image paths (no CDN) FAIL — calendar.html has 3 Squarespace CDN refs
3. Nav label consistency FAIL — index.html says "Calendar", rest say "Schedule"
4. Footer socials (IG+TT+FB) PASS
5. OG/Twitter meta tags PASS
6. Form labels PASS

Recommendation

NEEDS REVISION — 2 concrete fixes required before deploy-ready:

  1. calendar.html (site/calendar.html): Replace 3 Squarespace CDN image URLs with local assets/images/ equivalents. The logo path used on other pages should be reused here; the two event images need local copies or appropriate local fallbacks.
  2. index.html (site/index.html): Change nav link label from "Calendar" to "Schedule" in both the desktop nav and the mobile overlay, to match the other three pages.

Once those two fixes land, project is deploy-ready.