Skip to content

Enable 2026 programapi loading#1559

Merged
egeakman merged 5 commits intoep2026from
ep2026-programme
May 2, 2026
Merged

Enable 2026 programapi loading#1559
egeakman merged 5 commits intoep2026from
ep2026-programme

Conversation

@egeakman
Copy link
Copy Markdown
Member

@egeakman egeakman commented Apr 17, 2026

/sessions
/speakers
/session/<name>
/speaker/<name>

@egeakman egeakman requested review from artcz and clytaemnestra April 17, 2026 14:57
@egeakman egeakman added the DO-NOT-MERGE Not ready to be merged yet label Apr 17, 2026
@egeakman egeakman requested a review from hypha April 17, 2026 14:59
@europython-society
Copy link
Copy Markdown
Member

europython-society commented Apr 17, 2026

Preview available

Key Value
url https://ep2026-programme.ep-preview.click
last update 2026-05-02T02:28:32.021Z

@egeakman egeakman removed the DO-NOT-MERGE Not ready to be merged yet label Apr 22, 2026
@egeakman egeakman requested review from cmaureir and Copilot April 22, 2026 19:57
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Enables the 2026 programme API-backed content collections and exposes the Sessions/Speakers pages through navigation and page whitelisting.

Changes:

  • Add /sessions and /speakers links under the “Programme” header nav.
  • Add /sessions and /speakers to the enabled pages whitelist.
  • Re-enable remote loading of sessions/speakers data in astro:content collections via loadData.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
src/data/links.json Adds “List of Sessions” and “List of Speakers” to the Programme nav items.
src/data/enabledPages.json Whitelists /sessions and /speakers.
src/content/config.ts Switches speakers/sessions collections back to loading from the configured programme APIs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/data/links.json
Comment thread src/content/config.ts
Comment thread src/content/config.ts Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@egeakman egeakman added the DO-NOT-MERGE Not ready to be merged yet label Apr 22, 2026
@egeakman egeakman requested a review from dariagrudzien April 30, 2026 02:58
@hypha
Copy link
Copy Markdown
Contributor

hypha commented May 1, 2026

hey hey looks good and thanks Ege! Just one small question:
In the https://ep2026-programme.ep-preview.click/speakers/, do we know why there are these empty slots on this particular row?
screenshot of the speakers page showing one row having only 1 avatar instead of 5 like the other rows

@egeakman
Copy link
Copy Markdown
Member Author

egeakman commented May 2, 2026

hey hey looks good and thanks Ege! Just one small question: In the https://ep2026-programme.ep-preview.click/speakers/, do we know why there are these empty slots on this particular row? screenshot of the speakers page showing one row having only 1 avatar instead of 5 like the other rows

Hey, thanks for reviewing. I don't know why, but I don't think it's related to the actual data. Let's try to fix it in another PR.

@egeakman egeakman merged commit 4889c47 into ep2026 May 2, 2026
7 checks passed
@egeakman egeakman deleted the ep2026-programme branch May 2, 2026 02:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DO-NOT-MERGE Not ready to be merged yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants