Skip to content

Conversation

@JakeGinnivan
Copy link
Contributor

No description provided.

@JakeGinnivan JakeGinnivan requested a review from Copilot September 2, 2025 01:02
Copy link
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

This PR implements a comprehensive sponsor management system and improves the website's layout consistency. The main purpose is to add sponsor pages and improve the overall layout structure of the website.

Key changes include:

  • Migrating from data() to new Response() with JSON.stringify for better API response handling
  • Creating a new standardized page layout system with PageLayout and related components
  • Adding comprehensive sponsor display functionality with dedicated routes and historical sponsor data

Reviewed Changes

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

Show a summary per file
File Description
app/routes/app-*.tsx Migrated API responses from data() to new Response() with proper JSON handling
app/routes/_layout.sponsors.($year).tsx New sponsor showcase page with year-based navigation
app/routes/_layout.agenda.($year).tsx Enhanced agenda layout with improved responsive design
app/routes/_layout.tsx & _layout.$.tsx Updated to use new standardized layout components
app/components/page-layout.tsx New reusable layout components for consistent page structure
app/components/hero/*.tsx & header/header.tsx Updated to use new layout system with improved navigation
app/config/years/*.server.ts Updated sponsor configurations with proper dark/light mode logo paths
website-content/pages/faq.mdx Streamlined FAQ content removing outdated information
scripts/add-sponsor.mjs Enhanced sponsor management tool with logo processing capabilities
package.json Added dependencies for sponsor management functionality
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@JakeGinnivan JakeGinnivan merged commit 0e2dd2b into main Sep 2, 2025
1 check passed
@JakeGinnivan JakeGinnivan deleted the feature/sponsor-love branch September 2, 2025 01:06
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