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

feat: Update to matchmaking rewrite #11

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

DaniElectra
Copy link
Member

Depends on PretendoNetwork/nex-protocols-common-go#45

Partially solves #3

Changes:

Also includes partial support for communities. They can be created and you can join one, but no statistics will be shown since legacy Ranking isn't implemented.

Also includes partial support for communities. They can be created and
you can join one, but no statistics will be shown since legacy Ranking
isn't implemented.

Partially solves #3
@Anonymous941
Copy link

Any updates on merging this? Either way, I've been waiting for months for this to be made, tysm for writing it!

@ashquarky
Copy link
Member

I think this shouldn't be merged right now, since while the code is good, it exposes some underlying bugs in nex-go that result in widespread connection issues (we've been seeing this on Wii U). We had to revert it in Splatoon for this reason: PretendoNetwork/splatoon#12 (comment)

This PR should be mergeable once that underlying nex-go issue is identified and fixed.

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.

3 participants