Skip to content

Conversation

Chesblaw
Copy link

@Chesblaw Chesblaw commented Sep 4, 2025

Updated the endpoints under route
/events/ to /events/coursebook/
/astra/ to /events/astra/
/mazevo/ to /events/mazevo/

@mikehquan19
Copy link
Contributor

Hi @Chesblaw, thank you for the effort, but there are a couple issues:

  • What I'm really looking for is that you move all the controller functions into the same file controllers/events.go and all the routes into routes/events.go, not just renaming of the endpoints.

  • The build fails in your PR so you may wanna double-check your work.

@mikehquan19
Copy link
Contributor

@Chesblaw , any updates on this?

@Chesblaw
Copy link
Author

Hey yea I'll open PR but i lefted the utils, evaluation, storage there is no route for them

@mikehquan19 mikehquan19 closed this Oct 3, 2025
@mikehquan19 mikehquan19 reopened this Oct 3, 2025
@mikehquan19
Copy link
Contributor

@Chesblaw , You should only group the events routes, leaving out everything else.

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