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

Enable multiple characters from the same account #1492

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

Conversation

thecatontheflat
Copy link
Contributor

No description provided.

@kamronbatman
Copy link
Contributor

I am not sure if that is enough to make this work. Sagas added a similar feature and they had a problem with CUO where it somehow got MUO to log in two NetStates to the same player. So we need specific checks for that.

@thecatontheflat
Copy link
Contributor Author

thecatontheflat commented Sep 5, 2023

image

This is the error CUO received on the logged in character when trying to login into the same character with the current implementation. It logs out the old character and logs in the new one

@kamronbatman
Copy link
Contributor

image This is the error CUO received on the logged in character when trying to login into the same character with the current implementation. It logs out the old character and logs in the new one

Yeah that is the happy path, but when CUO is modified to allow multiple characters, it somehow showed the player twice with a small delay between.

@kamronbatman
Copy link
Contributor

kamronbatman commented Apr 20, 2024

I did some testing on this. Looks like the CUO bug is an old one.

@thecatontheflat Let's modify this to use a number indicating the max amount of characters that can be logged in at once.

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.

None yet

2 participants