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: miscellanous NanoSocket bits and pieces, add Mirage Standalone support #1106

Merged
merged 4 commits into from
Oct 8, 2022

Conversation

SoftwareGuy
Copy link
Contributor

@SoftwareGuy SoftwareGuy commented Oct 8, 2022

This is a multi-commit PR that addresses a few things:

  1. Renamed some variables (ie isWebgl is now isWebGL). This is more just for cosmetic appearance, and these were used internally and not outside of the Mirage Sockets' layer.
  2. Introduced NanoSockets compatibility for Mirage Standalone. Had to remove #if UNITY_EDITOR || UNITY_STANDALONE defines in the NanoSockets source files or the Mirage Standalone project would complain.
  3. Patched isDesktop to point to a boolean-returning function instead of a expression for Mirage Standalone compatibility.
  4. Rewrote/Reworked some tooltip(s) and error message(s).

Tested briefly on Windows instance of the Unity Editor version 2020.3.38. Shouldn't break anything, I hope.

@SoftwareGuy SoftwareGuy merged commit 1b7e097 into master Oct 8, 2022
github-actions bot pushed a commit that referenced this pull request Oct 8, 2022
## [128.4.1](v128.4.0...v128.4.1) (2022-10-08)

### Bug Fixes

* miscellanous NanoSocket bits and pieces, add Mirage Standalone support ([#1106](#1106)) ([1b7e097](1b7e097))
@github-actions
Copy link
Contributor

github-actions bot commented Oct 8, 2022

🎉 This PR is included in version 128.4.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@paulpach paulpach deleted the nanosockets-fixing branch October 18, 2022 02:45
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

1 participant