SyntaxMeets icon indicating copy to clipboard operation
SyntaxMeets copied to clipboard

Published 20 hours ago • verified publisher icon kothariji

Move Socket Outside

Open nishant23122000 opened this issue 3 years ago • 3 comments

Describe the bug I think we should move all socket.io code outside of index.js in the backend because there is a bunch of socket.io events happening in index.js.

To Reproduce

Expected result We can create an object of the socket in the socket.js file which we store in Utils folder So I think it's easy to understand index.js code and also code will be refactored in proper structure

Actual result

Screenshots

Are you willing to contribute to this issue? Yes

Desktop (please complete the following information):

  • OS: Windows 10
  • Browser Chrome
  • Version 89.0.4389.114

Smartphone (please complete the following information):

Additional context

nishant23122000 avatar Apr 10 '21 05:04 nishant23122000

Your point seems realistic. @nishant23122000 this will help us in extending the socket in the future @kothariji

Vishv07 avatar Apr 10 '21 06:04 Vishv07

@Vishv07 Yes

nishant23122000 avatar Apr 10 '21 07:04 nishant23122000

Hi @nishant23122000 , First of all, thank you so much for taking out the time to work on this issue.

Definitely 🙌, Please go forward to work on this issue. I am assigning this Issue to you 👍

And last but not least, Keep Contributing 🙏🔥

kothariji avatar Apr 10 '21 12:04 kothariji