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.

Sign up for GitHub

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

Jump to bottom

[Bug]: Missing message for Player Already Connected. #3311

Closed
Phergus opened this issue Jan 2, 2022 · 1 comment
Closed

[Bug]: Missing message for Player Already Connected. #3311

Phergus opened this issue Jan 2, 2022 · 1 comment
Assignees
@Phergus
Labels
bug tested This issue has been QA tested by someone other than the developer.

Comments

@Phergus
Copy link
Contributor

Phergus commented Jan 2, 2022

Describe the Bug

If a player tries to connect a second time the error dialog shows the key and not that message.

To Reproduce

  1. Start server
  2. Start 2nd client and attempt to connect with same username
  3. Get error message (see screenshot)

Expected Behaviour

Expect to see proper error message.

Screenshots

unknown

MapTool Info

MT 1.11.4

Desktop

Any

Additional Context

Reported by Kolifarr on Discord.
https://discord.com/channels/296230822262865920/296657960720007169/927211287203692554
@Phergus Phergus added the bug label Jan 2, 2022
@Phergus Phergus self-assigned this Jan 2, 2022
@Phergus Phergus mentioned this issue Jan 2, 2022
Merged
@kwvanderlinde
Copy link
Collaborator

Verified that the message "That player name is already connected." is now displayed instead of a translation key.

@kwvanderlinde kwvanderlinde added the tested This issue has been QA tested by someone other than the developer. label Jun 14, 2022
@Phergus Phergus closed this as completed Jul 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Phergus Phergus

Labels
bug tested This issue has been QA tested by someone other than the developer.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@kwvanderlinde @Phergus