-
-
Notifications
You must be signed in to change notification settings - Fork 93
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
Randomize all maps when there is only one player #583
Conversation
When playerCount = 1, will get all maps, not only maps with maxplayers = 2
Nightly build for this pull request:
|
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.
Note: please address the review comments before your PR gets merged
@Flactine This is a reminder that please address the review comment since we want to merge this PR. |
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.
Good
You need to address the review comment, which means you need to pushing more commits so that your codes does not violate the requirements. It's not hitting the "Resolve conversation" button -- it just sent the information that you have addressed the comment, rather than let GitHub automatically solve it. |
Add a line break after }
@Flactine After you have addressed all concerns, please hit each "Resolve conversation" button and then re-request reviews. See the screenshot below. |
When the number of players is 1, the random map will be assigned to any map, regardless of the max players of the map.
Used to randomly check maps.