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

Integrated Socket.io | Updated Readme.md #146

Open
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

TheAdich
Copy link

Resolves #126

PR Fixes:

  • 1 Integrated Socket.io
  • 2 Updated Readme.md on how to setup the server

Checklist before requesting a review

  • I have pull latest changes from main branch
  • I have tested the changes locally
  • I have run npm run lint:fix locally
  • I have performed a self-review of my code
  • I assure there is no similar/duplicate pull request regarding same issue

Copy link

vercel bot commented Oct 11, 2024

@TheAdich is attempting to deploy a commit to the Dhanush Nehru's projects Team on Vercel.

A member of the Team first needs to authorize it.

@TheAdich
Copy link
Author

screen-capture.webm

@TheAdich
Copy link
Author

Screenshot 2024-10-11 193901
Screenshot 2024-10-11 193922
On
Screenshot 2024-10-11 193947
On render you just have to create an account choose web service choose repo in which your backend code is written specify its path like in my case i had a seperate repo for my backend and in the build command write npm install and in the start command node index.js and click on deploy
If you want to run your project locally then no need to configure to render just start the server by giving your own mongodb connection string as shown in video

@DhanushNehru
Copy link
Owner

Hi @TheAdich please resolve merge conflicts

@TheAdich
Copy link
Author

Github is saying no merge conflicts with the base branch currently

Copy link

vercel bot commented Oct 16, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
custom-code-editor ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 16, 2024 6:57am

@DhanushNehru
Copy link
Owner

Hi @TheAdich it seems mongo db has to be hosted somewhere inorder to incorporate your changes. Is there any free version available.

Also since its deployed in vercel. How we can setup things ??

The Render setup has been done from my end. Now MongoDB connection string not sure what to be done !

server/index.js Outdated
app.use('/api/code',codeRouter);


mongoose.connect("mongodb+srv://testing_node:[email protected]/customDb?retryWrites=true&w=majority&appName=Cluster0")
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also why the mongo connection string is hard coded ?
It could be from .env

@TheAdich
Copy link
Author

TheAdich commented Oct 16, 2024 via email

@TheAdich
Copy link
Author

TheAdich commented Oct 16, 2024 via email

@TheAdich TheAdich reopened this Oct 16, 2024
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.

Integrating Socket.io for real time coding with peers
2 participants