-
Notifications
You must be signed in to change notification settings - Fork 219
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
expand disconnect reasons #815
base: master
Are you sure you want to change the base?
Conversation
add the new disconnect reasons
Should we use piqueserver/aosprotocol#15 instead of adding more fixed strings? |
There's a few things with that:
|
// exceeded. | ||
// we need to check other usages | ||
return _Tr("NetClient", "You were kicked from this server."); | ||
case 2: return _Tr("NetClient", "You have too many connections to this server."); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The word “you” might be inappropriate here because it’s not necessarily “you” who has the connections (e.g., other people behind a NAT).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had similar concerns but I was unsure how to phrase it instead. "Your IP" might be correct at the moment, but:
- people, in general, are not that familiar with where their IP comes from, especially in larger networks and with the increased prevalence of CGNATs
- this is a technical detail and might change in the future
Perhaps "Your location" or "Your network" is a good solution?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems like "Your network's public address has too many connections to this server" would be the best solution, even though some users may not understand it, I believe it's best to deliver the most accurate message.
Maybe a text below with additional info could be added including the common reasons this might be happening.
add the new disconnect reasons