Feature/deployment swey - #238
Conversation
Demo 2 Merge
…to feature/deployment-swey
morganthegirlboss
left a comment
morganthegirlboss
left a comment
There was a problem hiding this comment.
Looked over this PR as requested of me and in terms of the work I do understand, everything looks correct and imports we shouldn't be using seem to have all been removed
There was a problem hiding this comment.
Why were these changes made for this file?
There was a problem hiding this comment.
becuase we're using an nginx proxy we don't need to spcify the websocket url, nginx handles the routing for us
There was a problem hiding this comment.
And this is only for deployment? Or is it needed for prod?
nsmzoneli
left a comment
nsmzoneli
left a comment
There was a problem hiding this comment.
Two players don't seem to be able to find one another when they queue. Is this intended behaviour? Please advise
I'm not sure what you mean. Is the matching not working on your machine? |
Yes, I followed the link, signed in two players, but they are not matching |
Signed-off-by: Swelihle Makhathini <u24584585@tuks.co.za>
…E-2026/CodeClash into feature/deployment-swey
Codecov Report❌ Patch coverage is 📢 Thoughts on this report? Let us know! |
|
3 participants
What does this PR do?
setups production deployment for the application
Related issue
#222
How to test
navigate to the url https://www.codeclash.co.za/ and run through the app as a user