Skip to content

Add FitTrybe dataset site - #56

Open
franschezco wants to merge 2 commits into
openactive:masterfrom
franschezco:patch-1
Open

franschezco wants to merge 2 commits into
openactive:masterfrom
franschezco:patch-1

Conversation

@franschezco

@franschezco franschezco commented Aug 17, 2026 •

Copy link
Copy Markdown

Publishing Checklist

I confirm the following:

Maintenance Checklist

Our organisation is committed to:

  • Ensuring activity providers are aware of the OpenActive initiative on an ongoing basis
  • Providing high quality opportunity data that meets user needs, as specified in OpenActive's Data Quality reporting framework
  • Resolving issues that are raised on the GitHub Issues Board(s)

@franschezco

Copy link
Copy Markdown
Author

checked

@franschezco franschezco reopened this Aug 17, 2026
@hkir-dev
hkir-dev requested review from hkir-dev and howaskew August 17, 2026 08:25
@hkir-dev

hkir-dev commented Aug 18, 2026 •

Copy link
Copy Markdown
Contributor

Thank you for your interest in Open Active. There are a few points to address before we can merge this:

1- superEvent and subEvent references point at the wrong records

In the ScheduledSession feed, superEvent is built from the ScheduledSession's own UUID:

"@id":        ".../openactive-scheduled-sessions?id=082029a1-3d96-43cc-aa13-c7b3739dc2b0"
"superEvent": ".../openactive-session-series?id=082029a1-3d96-43cc-aa13-c7b3739dc2b0"   ← same UUID

superEvent must be the @id of the parent SessionSeries, not a session-series URL constructed from the child's ID. This issue also exists in the SessionSeries subEvent references.

2- Please remove subEvent from the SessionSeries feed
Split feeds are the recommended configuration, and the combined patterns are no longer recommended for new implementations.

The expected shape would be:

// SessionSeries feed — no subEvent
{ "@type": "SessionSeries", "@id": "https://.../session-series/b8fd16d2-...", ... }

// ScheduledSession feed
{ "@type": "ScheduledSession",
  "@id": "https://.../scheduled-sessions/082029a1-...",
  "superEvent": "https://.../session-series/b8fd16d2-...", ... }

3- Suggestion: use PostalAddress for the address
Rather than putting the address in description, it could be represented as a structured PostalAddress:

"address": {
  "@type": "PostalAddress",
  "streetAddress": "East St",
  "addressLocality": "Epsom",
  "addressRegion": "Surrey",
  "postalCode": "KT17 1BN",
  "addressCountry": "GB"
}

4- Please also check the startDate/endDate values
The subEvents currently included in the SesssionSeries feed (which we aim to remove) are dated 2026-06-27, while the corresponding ScheduledSessionss are dated 2026-02-21. Could you please confirm whether these dates are correct?

Thanks again for the contribution!

This branch has not been deployed

No deployments
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