Skip to content

Conversation

@alexdrenea
Copy link

Added a flag in SharingStage to enable server auto discovery.
Uses the first server that responds to connect to.
Fixes #172

Added a flag in SharingStage to enable server autodiscovery. Uses the first server that responds to connect to.
/// <summary>
/// Provides callbacks when server is discovered or lost.
/// </summary>
private DiscoveryClientAdapter listener;
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NIT: discoveryClientListener

@alexdrenea
Copy link
Author

Fixed... Sorry for the delay.

@alexdrenea
Copy link
Author

Any updates on this PR? I've been on an extended vacation (and will still be out in the coming week) but wanted to check in and see if there's anything missing.

@jwittner
Copy link
Member

@alexdrenea I'll try to check this out tomorrow. Sorry for the long delay on this one.

@jwittner
Copy link
Member

Worked great for me!

@jwittner jwittner merged commit a0b6193 into microsoft:master Sep 15, 2016
jwittner pushed a commit to jwittner/MixedRealityToolkit-Unity that referenced this pull request Nov 16, 2016
 - Added BaseGestureInteractive controls and test scene.

Related work items: microsoft#177
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.

4 participants