Skip to content

Replace excerpts with new summary field#130

Merged
alexwolson merged 1 commit into
mainfrom
replace-excerpt-with-summary
Jun 25, 2026
Merged

Replace excerpts with new summary field#130
alexwolson merged 1 commit into
mainfrom
replace-excerpt-with-summary

Conversation

@Khasir

@Khasir Khasir commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator

Description

  • This PR replaces the excerpt field with a new summary field.
  • Currently, code in various pages checks if excerpt exists and then displays it. However, Jekyll creates an excerpt of the first paragraph by default, defeating the purpose of this check, and leading to content duplication. I believe the intent of the check is to only provide the excerpt if we've explicitly written one.

How to Test

  • Built locally and checked a sample of impacted pages for issues

Screenshots or Recording (if applicable)

Before - note the duplicated text, as no excerpt was provided:

image

After:

image

To Do

  • After merging this PR, will have to update the archives submodule so that records with excerpt use the new summary field. This will lead to temporary loss of visible excerpts on the website until the archives are updated.
  • I will update archives as needed.

@github-actions

Copy link
Copy Markdown
Contributor

Pa11y Accessibility Report

All pages passed — 0 warning(s) (WCAG2AA)


Homepage

✅ No issues found.


Event

✅ No issues found.


Project

✅ No issues found.


Person

✅ No issues found.


Organization

✅ No issues found.


Venue

✅ No issues found.


Resource

✅ No issues found.


Tag

✅ No issues found.


Category

✅ No issues found.

@Khasir
Khasir requested a review from alexwolson June 11, 2026 18:10
@alexwolson
alexwolson merged commit 3dd5403 into main Jun 25, 2026
6 checks passed
@alexwolson
alexwolson deleted the replace-excerpt-with-summary branch June 25, 2026 12:29
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.

2 participants