Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Separate API endpoints and Socket endpoints into different files and load in index.ts #219

Closed
AlexanderWangY opened this issue Apr 13, 2024 · 1 comment
Assignees
Labels
Backend📡 Issues corresponding to the backend/API. Medium You should have some familiarity with our codebase to work on this issue Refactor Rewriting functions, changing names, improving code readability, reducing complexity

Comments

@AlexanderWangY
Copy link
Collaborator

As of right now, our backend's endpoints are all stored in index.ts which is a monolith of code which gets pretty confusing. We'd like you to help us sort them our into separate files, one for API related functions and one for Socket related functions!

Some tips:

  • Use the router functionality given by expressJS and app.use it in the index.ts (possibly?)
  • Make initializing sockets a function that is called once in index.ts on startup?

Move other files into their respective folder's and make sure that if something breaks, we can pinpoint exactly whether it was in the sockets or api!

@AlexanderWangY AlexanderWangY added Backend📡 Issues corresponding to the backend/API. Medium You should have some familiarity with our codebase to work on this issue Refactor Rewriting functions, changing names, improving code readability, reducing complexity labels Apr 13, 2024
@fmcubium fmcubium self-assigned this Apr 13, 2024
@fmcubium
Copy link
Collaborator

Taking on this task!

@AlexanderWangY AlexanderWangY closed this as not planned Won't fix, can't repro, duplicate, stale Aug 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Backend📡 Issues corresponding to the backend/API. Medium You should have some familiarity with our codebase to work on this issue Refactor Rewriting functions, changing names, improving code readability, reducing complexity
Projects
Development

No branches or pull requests

2 participants