Skip to content

Conversation

@Askaholic
Copy link
Collaborator

Fixes #267

Adds some additional tests for command_game_join

@Rackover
Copy link
Member

Hey !
Whoever you are, thanks for all those awesome contributions you made.
Would you mind joining our slack so that we can communicate more easily ?
Due to christmas-end-year period I won't be able to review and merge those PRs very quickly, although they look promising. But I certainly appreciate them and will make sure to look into it as soon as I can 🙂

@Askaholic
Copy link
Collaborator Author

Yea I tried to request access by sending someone on the forums a message, but it was being blocked as spam.

@Geosearchef
Copy link
Member

I sent you a slack invite via the email included as author in your git commits.

@Rackover Rackover merged commit 9a1d822 into FAForever:develop Dec 29, 2018
@Askaholic Askaholic deleted the issue/#267-uid-as-str branch December 29, 2018 23:22
Askaholic added a commit to Askaholic/server that referenced this pull request Dec 30, 2018
…r#379)

* Cast uid and port to int even if they are sent as strings

* Added some more tests for game_join
Brutus5000 pushed a commit that referenced this pull request Jun 24, 2019
* Cast uid and port to int even if they are sent as strings

* Added some more tests for game_join
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