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

fix: adding ability to start the Client & Server from interfaces #1079

Merged
merged 3 commits into from
May 24, 2022
Merged

fix: adding ability to start the Client & Server from interfaces #1079

merged 3 commits into from
May 24, 2022

Conversation

Soren025
Copy link
Contributor

This PR stems from this conversation. It adds the ability to start the server from the INetworkServer and connect from the INetworkClient.

Starting the host is also possible from the interfaces, however it required making several methods public instead of internal to allow for the interfaces to decouple the NetworkServer and NetworkClient. Let me know if this is ok.

@Soren025
Copy link
Contributor Author

Reverted the support of host mode in the interface methods to remove the need to expose internal methods in NetworkServer and NetworkClient.

@James-Frowen James-Frowen changed the title Add Ability to Start the Client & Server From Interfaces fix: adding ability to start the Client & Server from interfaces May 24, 2022
@James-Frowen James-Frowen merged commit 61e8d6b into MirageNet:master May 24, 2022
github-actions bot pushed a commit that referenced this pull request May 24, 2022
## [123.1.3](v123.1.2...v123.1.3) (2022-05-24)

### Bug Fixes

* adding ability to start the Client & Server from interfaces ([#1079](#1079)) ([61e8d6b](61e8d6b))
@github-actions
Copy link
Contributor

🎉 This PR is included in version 123.1.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants