Skip to content

Refactored the event.go handler's file#136

Closed
maernest04 wants to merge 1 commit into
mainfrom
ernest_ma/refactor_handlers_event_file
Closed

Refactored the event.go handler's file#136
maernest04 wants to merge 1 commit into
mainfrom
ernest_ma/refactor_handlers_event_file

Conversation

@maernest04

Copy link
Copy Markdown
Collaborator

PR for #118

Problem

For the handlers/event.go folder, the folder was cluttered with every event API such as event reading and writing, getting attendee information, registration and waitlist, and admin registration dashboard information. This made the file way too big and cluttered.

Solution

Separate the handlers/event.go file into separate files, using the same package name "handlers" but separating each event APIs into their own file.

@steeevin88 steeevin88 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.

can we make a new event folder in here? and then simplify the file names, ex.

handler.go
read.go
write.go
registration.go
admin_registration.go
attendee.go
waitlist.go

@steeevin88 steeevin88 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.

we should also split the test file too

@maernest04 maernest04 closed this May 5, 2026
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