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

Refactor socket.io endpoints to seperate files and add better error checking #288

Open
h1divp opened this issue Sep 14, 2024 · 1 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

Comments

@h1divp
Copy link
Collaborator

h1divp commented Sep 14, 2024

Like the recent express api refactor #270 , we would like to also move the socket endpoints in server/index.ts to files in their own folder.

For this, please create a folder called server/src/sockets and find a way to break up the socket connections. If you are familiar with socket.io/web sockets and know about related error handling best practices, we would really appreciate your help with this issue. If not, please familiarize yourself with websocket documentation while working on this issue.

@h1divp h1divp 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 Sep 14, 2024
@AaronGibson2
Copy link
Contributor

I would like to work on this.

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
None yet
Development

No branches or pull requests

2 participants