Skip to content

Comments

Add match header/stats URLs to DCAR data model #28610

Merged
marjisound merged 3 commits intomainfrom
add-match-header-and-stats-urls-to-dcar-model
Feb 20, 2026
Merged

Add match header/stats URLs to DCAR data model #28610
marjisound merged 3 commits intomainfrom
add-match-header-and-stats-urls-to-dcar-model

Conversation

@marjisound
Copy link
Contributor

@marjisound marjisound commented Feb 16, 2026

What does this change?

This PR:

  • Adds a new Sport API route for match-stats-summary:
    /football/api/match-stats-summary/:year/:month/:day/:home/:away.json
    This endpoint returns a summary of match stats and will be used for the match stats section on football live/dead blog pages.
  • Extends the DCAR article data model (match reports and live/dead blogs) to include:
    • matchHeaderUrl
    • matchStatsUrl
  • Updates URL selection logic based on pageType:
    • live/dead blog -> match-stats-summary
    • none blog article -> match-stats

NOTE:

The existing field in DCAR data model matchUrl will be deprecated and removed after the migration

Screenshots

DCAR data model for articles:

Before After
before after

Data model for match-stats-summary endpoint:

image

Fixes part of guardian/dotcom-rendering#15174

@github-actions
Copy link
Contributor

github-actions bot commented Feb 16, 2026

Base automatically changed from add-match-stats-json-endpoint to main February 17, 2026 16:08
@marjisound marjisound force-pushed the add-match-header-and-stats-urls-to-dcar-model branch from 9d489c8 to 3cc43ec Compare February 17, 2026 16:28
@marjisound marjisound changed the title Add matchHeaderUrl and matchStatsUrl to DCAR data model Add match header/stats URLs to DCAR data model and match-stats-summary route Feb 17, 2026
@marjisound marjisound changed the title Add match header/stats URLs to DCAR data model and match-stats-summary route Add match header/stats URLs to DCAR data model Feb 17, 2026
@marjisound marjisound added the feature Departmental tracking: work on a new feature label Feb 17, 2026
@marjisound marjisound self-assigned this Feb 17, 2026
@marjisound marjisound marked this pull request as ready for review February 17, 2026 17:52
@marjisound marjisound requested a review from a team as a code owner February 17, 2026 17:52
@marjisound marjisound merged commit 78362ed into main Feb 20, 2026
17 checks passed
@marjisound marjisound deleted the add-match-header-and-stats-urls-to-dcar-model branch February 20, 2026 12:47
@gu-prout
Copy link

gu-prout bot commented Feb 20, 2026

Seen on ADMIN-PROD (merged by @marjisound 8 minutes and 59 seconds ago)

@gu-prout
Copy link

gu-prout bot commented Feb 20, 2026

Seen on FRONTS-PROD (merged by @marjisound 9 minutes and 18 seconds ago)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature Departmental tracking: work on a new feature Seen-on-ADMIN-PROD Seen-on-FRONTS-PROD

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants