Skip to content

Update events page#696

Open
davehorsfall wants to merge 5 commits into
mainfrom
feat/events-update
Open

Update events page#696
davehorsfall wants to merge 5 commits into
mainfrom
feat/events-update

Conversation

@davehorsfall

@davehorsfall davehorsfall commented Apr 17, 2026

Copy link
Copy Markdown
Member

Description

This PR updates the existing events page, populating each event with more details.

image

Fixes # (issue)

Type of change

  • Documentation (non-breaking change that adds or improves the documentation)
  • New feature (non-breaking change which adds functionality)
  • Optimization (non-breaking, back-end change that speeds up the code)
  • Technical work (non-breaking, change which is work as part of a new feature)
  • Bug fix (non-breaking change which fixes an issue)
  • Breaking change (whatever its nature)

Key checklist

  • All tests pass (eg. python -m pytest)
  • The documentation builds and looks OK (eg. mkdocs serve)
  • Pre-commit hooks run successfully (eg. pre-commit run --all-files)

Further checks

  • Code is commented, particularly in hard-to-understand areas
  • Tests added or an issue has been opened to tackle that in the future. (Indicate issue here: # (issue))

Copilot AI review requested due to automatic review settings April 17, 2026 14:46
@davehorsfall davehorsfall changed the title Feat/events update Update events page Apr 17, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the Events feature to support richer event metadata (location + external links) sourced from an expanded CSV schema, and adjusts navigation links accordingly.

Changes:

  • Extend data/events.csv schema and update EventsPageView to include location/event/blog fields and optional images.
  • Update the Events page template to render the new fields and handle missing images.
  • Tweak navigation/footer links and add a new event image asset.

Reviewed changes

Copilot reviewed 5 out of 9 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
main/views/page_views.py Reads new CSV columns into the Events context and sorts events by start date.
main/templates/main/pages/events.html Renders new event fields (location/event/blog) and adds a “no image” placeholder.
data/events.csv Replaces the old CSV layout with a new header + enriched event records.
main/templates/main/snippets/navbar.html Updates the Framework dropdown label for the roles page.
main/templates/main/snippets/footer.html Adds a Framework “Overview” link in the footer.
main/static/assets/img/events/DisCouRSE -team-culture.jpg Adds a new event image asset.

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

Comment thread data/events.csv Outdated
Comment thread main/templates/main/snippets/navbar.html Outdated
Comment thread main/templates/main/pages/events.html Outdated
Comment thread main/templates/main/pages/events.html Outdated
Comment thread main/views/page_views.py Outdated
Comment thread main/views/page_views.py

@anenadic anenadic left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is looking good to me, would nice to add slides we used at events and maybe other docs but this can also be added progressively.

@AdrianDAlessandro AdrianDAlessandro left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Overall, this is good. I still think that longer term this should be moved to the database, or pulled from a separate location, like how the governance and licensing pages have been done.

Again, some changes required to make tests pass, and some good suggestions from copilot to address

Comment thread main/static/assets/img/events/DisCouRSE -team-culture.jpg Outdated
Comment thread data/events.csv Outdated
Comment thread main/templates/main/pages/events.html Outdated
Comment thread main/templates/main/pages/events.html Outdated
Comment thread main/templates/main/pages/events.html Outdated
Comment thread main/templates/main/snippets/navbar.html Outdated
Comment thread main/views/page_views.py Outdated
@codecov

codecov Bot commented Apr 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Base automatically changed from feat/governance to main April 24, 2026 15:44
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.

4 participants