Skip to content

Add anonymous post-meetup feedback survey at /feedback/#78

Open
alexwolson wants to merge 20 commits intomainfrom
feature/feedback-survey
Open

Add anonymous post-meetup feedback survey at /feedback/#78
alexwolson wants to merge 20 commits intomainfrom
feature/feedback-survey

Conversation

@alexwolson
Copy link
Collaborator

@alexwolson alexwolson commented Feb 24, 2026 •
edited
Loading

Summary

  • Adds a new page at /feedback/ with a fully anonymous post-meetup feedback form
  • Submissions are written as JSON files to the CivicTechTO/feedback repo via the GitHub Contents API using a scoped PAT
  • Form uses progressive disclosure: core questions are always visible, first-timer and online-attendee sections appear conditionally, equity and open-ended questions live in optional expanders
  • Styled with a custom "The Public Square" civic editorial design — DM Serif Display + DM Sans, warm cream cards, Toronto red accents, pill toggles and numbered rating buttons

What's included

Token injection (_config.yml, .gitignore, Makefile, pages.yml):

  • feedback_token config key injected at build time from FEEDBACK_TOKEN GitHub Actions secret
  • _config.local.yml pattern for local dev (gitignored); Makefile picks it up automatically

Feedback page (_pages/feedback.md):

  • Meetup selector (last 10 hacknights)
  • Attendance mode + first-time questions (pill toggles)
  • Overall rating (1–5 numbered buttons)
  • Conditional first-timer section (felt welcome, understood what was happening, would return)
  • Conditional online-attendee section (A/V quality, felt included)
  • Optional "Tell us more" expander (open text)
  • Optional "About you" expander (gender identity, age range, racialized, disability, belonging, topic relevance)
  • Client-side GitHub API submission with success/error states

CSS (assets/css/custom.css): inline .rating-row base style (page-specific overrides live in the page's own <style> block)

Before merging

  1. Create the CivicTechTO/feedback public repo (with a submissions/ directory placeholder)
  2. Generate a fine-grained PAT scoped to that repo with Contents: Read and write
  3. Add FEEDBACK_TOKEN to this repo's Actions secrets

Test plan

  • Build site locally with _config.local.yml containing feedback_token: <your-pat>
  • Visit /feedback/, select a meetup — submit button should enable
  • Select "In person" / "Online" — online section should slide in/out
  • Select "Yes, first time" / "I've been before" — first-timer section should slide in/out
  • Submit the form — verify a JSON file appears in CivicTechTO/feedback/submissions/
  • Confirm the form hides and success message appears

🤖 Generated with Claude Code

alexwolson and others added 13 commits February 24, 2026 08:13
@alexwolson
Add feedback survey design doc
a7eeeda
@alexwolson
Add feedback survey implementation plan
f31739c
@alexwolson @claude
Configure feedback form token injection for build
26c1a6c 
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@alexwolson
Fix missing trailing newline in .gitignore and quote token in CI inje…
68fc4ed 
…ction
@alexwolson
Add feedback page with form structure and all questions
636672d
@alexwolson @claude
Fix rating fieldset semantics, option whitespace, and alert roles
9ee7628 
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@alexwolson
Add inline rating row style for feedback form
b3f91a5
@alexwolson
Add conditional question visibility and submit-enable JS
cfa6fc6
@alexwolson
Add GitHub API submission to feedback form
989770d
@alexwolson @claude
Fix timestamp consistency and add parseInt radix in feedback form JS
b2a08fa 
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@alexwolson
Add aria-labelledby to conditional sections and update CSS header date
3c0bfee
@alexwolson @claude
Redesign feedback form with civic editorial aesthetic
e9eff25 
Replaces default Pico styling with a custom "The Public Square" design:
- DM Serif Display + DM Sans typography via Google Fonts
- Warm cream/white card layout with Toronto red (#C8102E) accent
- Radio groups converted to pill toggle buttons
- Rating scales converted to square numbered buttons with serif numerals
- Conditional sections use CSS max-height transitions (replaces hidden attr)
- Custom details/summary expanders with +/− toggle
- Intro text moved to front matter excerpt for clean layout rendering

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@alexwolson @claude
Replace meetup dropdown with Tuesday-only date picker
c4166e5 
Removes the dependency on the archives submodule for the feedback form.
Instead of a dropdown limited to meetups in the archives, uses a date
input defaulting to the most recent Tuesday. step=7 with a Tuesday min
restricts the browser calendar to Tuesdays; JS validates on change as a
fallback. Stores hacknight_date (ISO date string) instead of meetup_number
in the submitted JSON.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@alexwolson alexwolson marked this pull request as draft February 24, 2026 14:25
alexwolson and others added 7 commits February 24, 2026 09:28
@alexwolson @claude
Use Pico CSS variables for feedback form colours and remove custom fonts
8e96505 
Removes Google Fonts (DM Serif Display, DM Sans) and all custom --fb-*
colour variables. Replaces them with --pico-primary, --pico-card-background-color,
--pico-muted-*, etc. so the form matches the site's pumpkin theme and
respects dark/light mode automatically. Success/error states keep hardcoded
semantic colours (green/red) as Pico has no equivalent variables for these.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@alexwolson @claude
Add follow-up questions for maybe/no responses on would_return
747c8cd 
When a first-timer selects Maybe, asks "What's giving you pause?".
When they select No, asks "What put you off?". Both slide in inline
below the pill buttons. Response stored as would_return_reason in
the new_attendee JSON object.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@alexwolson @claude
Tone down would_return follow-up question labels
c0e2708 
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@alexwolson @claude
Unify form label typography for consistency
8b5612e 
All question labels/legends now share the same style: 0.875rem, 600
weight, --pico-color. Removes all-caps and muted-color treatment from
the date picker label and expander body labels, normalises the followup
label size, drops primary colour from the conditional section heading,
and fixes two broken --fb-muted variable references.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@alexwolson @claude
Make public storage of responses explicit and prominent
863dd90 
Adds a left-bordered notice banner at the top of the form and updates
the page excerpt subtitle to both state that responses are anonymous
but stored in a public GitHub repository.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@alexwolson @claude
Replace 5-point ratings with labelled 3-point scale, remove optional …
af76c22 
…labels, centre form

- Swap all seven 1–5 numeric rating scales for 3-option pill buttons with
  descriptive text labels (e.g. Not great / Good / Great), removing the
  need for hint text and making stored responses self-explanatory
- Fix button hover behaviour so rating buttons match pill buttons
- Fix grammatically awkward "Poor" option for AV quality question
- Remove "(optional)" labels from expander headings and privacy note —
  every field on the form is optional
- Centre the form wrapper on wider screens with margin: 0 auto

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@alexwolson @claude
Fix YAML syntax error in feedback token injection step
9f3c9fb 
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@alexwolson
Copy link
Collaborator Author

alexwolson commented Feb 25, 2026

(Failed to deploy as the branch is not allowed to deploy - build succeeded).

@alexwolson alexwolson marked this pull request as ready for review February 25, 2026 00:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Khasir Khasir Awaiting requested review from Khasir

@jamiepantling jamiepantling Awaiting requested review from jamiepantling

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@alexwolson