Skip to content

New SampleDeviceCollection#192

Merged
david-c-kline merged 23 commits into
microsoft:masterfrom
GregPettyjohn:master
Oct 17, 2016
Merged

New SampleDeviceCollection#192
david-c-kline merged 23 commits into
microsoft:masterfrom
GregPettyjohn:master

Conversation

@GregPettyjohn
Copy link
Copy Markdown
Contributor

Sample demonstrates a "vertical slice" of the WindowsDevicePortalWrapper. Provides a user interface to connect to and manage multiple devices. Including, renaming devices, rebooting devices, hooking up to performance events, etc. Provides an intuitive sign-in experience that clarifies the port setting and https setting based on the device type. Sample uses WPF + XAML and leverages the MVVM pattern with minimal code-behind. View models are designed to be reusable so that you can leverage them in your own code.

@msftclas
Copy link
Copy Markdown

Hi @GregPettyjohn, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!


It looks like you're a Microsoft contributor (Greg Pettyjohn). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

Comment thread .gitignore
*.user
*.userosscache
*.sln.docstates
sync.cmd
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

did I accidentally commit this file? sorry :(

Copy link
Copy Markdown
Contributor

@WilliamsJason WilliamsJason left a comment

Choose a reason for hiding this comment

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

I don't have a ton of time so I looked over this very quickly, but it looks good to me from what I saw. We should give other folks a bit of time to look over it but I'm happy to accept the PR in a day or two if nobody else has time to review it further.

}
}

/// <summary>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks like your rename to DefaultDevicePortalConnection got lost here. Also, I'll close the other PR that did this since that one is out-of-date.

@hpsin
Copy link
Copy Markdown
Contributor

hpsin commented Oct 17, 2016

Signing off on this as well. Only ran into a couple snags:

  • Connected to my IoT VM, but failed to get the device name. That shouldn't be related to your code though, had some exceptions from the wrapper in Get that failed.
  • Should this be in the Samples folder in the all up solution? It doesn't appear there today.

Couldn't get my HoloLens VM up and running, the storm over the weekend hosed it, but IoT, PC, and mobile all worked great otherwise.

@david-c-kline david-c-kline merged commit bb4bc56 into microsoft:master Oct 17, 2016
@GregPettyjohn
Copy link
Copy Markdown
Contributor Author

@hpsin
Hirsch, the IoT issue is a known issue ( see issue #180 )

This sample is a bit larger than the other samples and takes a dependency on Prism via NuGet. So it is probably easier to keep it in a solution by itself.

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.

5 participants