Open
Conversation
… buckets NOTE: ALL SERVER CODE IS NOW TARGETING LOCALLY HOSTED SERVERS. This is a large commit involving the initialization and connection of the matchmaker server to the regular mp server. All user communication will go through the mp server only. Additionally, the groundwork has been laid for all server communication from client to matchmaker and back again. The main piece that is missing now is the actual matchmaking logic. Currently, users are placed in a mmr-specific bucket upon joining the queue to simplify matchmaking. All ranked game-joining logic has been set in place. Once a condition is met, the matched players will all be emitted the appropriate lobby ID for their ranked game and the front-end logic should be working.
Still no matchmaking algorithm but all data is being passed properly and the game starts
ColinHarker
requested changes
Apr 6, 2022
| console.log("creating new connection") | ||
| socketRef.current = io.connect("https://generated-respected-python.glitch.me") | ||
| //socketRef.current = io.connect("https://generated-respected-python.glitch.me") | ||
| socketRef.current = io.connect("http://localhost:4000") |
Contributor
There was a problem hiding this comment.
Do we want to push this to develop?
|
|
||
| socketRef.current = io.connect("https://generated-respected-python.glitch.me") //LOCALHOST for local testing | ||
| //socketRef.current = io.connect("https://generated-respected-python.glitch.me") | ||
| socketRef.current = io.connect("http://localhost:4000") |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.