Skip to content

Conversation

@jzgom067
Copy link
Member

@jzgom067 jzgom067 commented Oct 20, 2025

This pull request fixes all the build errors being generated by ESLint.

That's it.

...and I gave the header side padding for mobile.

Copilot AI review requested due to automatic review settings October 20, 2025 02:37
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 pull request fixes ESLint build errors across the codebase by addressing type safety issues, removing unused variables and imports, and improving code quality. The changes focus on replacing any types with proper TypeScript types, cleaning up commented code, and fixing various linting violations.

Key changes:

  • Added proper TypeScript type definitions for API responses and component props
  • Removed unused variables, imports, and commented-out code
  • Fixed environment variable references from client-side to server-side variables
  • Corrected React Hook dependency arrays and ESLint violations

Reviewed Changes

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

Show a summary per file
File Description
next.config.ts Changed environment variable from NEXT_PUBLIC_API_URL to API_URL
example.env Updated environment variable name to match next.config.ts
app/_utils/fetch-data.tsx Added TypeScript type definitions for all API response types
app/_utils/format-api-error.tsx Replaced any type with proper ApiErrorResponse type
app/_utils/process-event-data.tsx Added type safety with EventDetailsResponse type
app/_utils/process-dashboard-data.tsx Added type definitions for dashboard data processing
app/_utils/submit-event.tsx Added EventSubmitJsonBody type and removed any casts
app/_utils/validate-data.tsx Removed unused error parameters in catch blocks
app/ui/layout/*.tsx Replaced any types with proper TypeScript interfaces
app/ui/components/**/*.tsx Removed unused imports, variables, and commented code
app/_lib/**/*.tsx Fixed hook dependency arrays and removed unused imports
app/[event-code]/**/page.tsx Centralized EventCodePageProps type definition

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

@jzgom067 jzgom067 requested a review from mirmirmirr October 20, 2025 02:49
Copy link
Member

@mirmirmirr mirmirmirr left a comment

Choose a reason for hiding this comment

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

👍

@mirmirmirr mirmirmirr merged commit bbd0ef9 into main-preview Oct 20, 2025
1 check passed
@mirmirmirr mirmirmirr deleted the build-error-fixes branch October 20, 2025 03:31
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.

3 participants