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

Anbare/toast additions 1801 #1759

Merged
merged 7 commits into from
Jan 23, 2018
Merged

Anbare/toast additions 1801 #1759

merged 7 commits into from
Jan 23, 2018

Conversation

andrewleader
Copy link
Contributor

@andrewleader andrewleader commented Jan 22, 2018

PR Type

New toast features for My People shoulder tap and upcoming toast telemetry dashboard.

[x] Feature

What is the current behavior?

These properties don't exist.

PR Checklist

Please check if your PR fulfills the following requirements:

  • Tested code with current supported SDKs
  • Docs have been added/updated which fit documentation template. (for bug fixes / features)
  • Sample in sample app has been added / updated (for bug fixes / features)
  • Tests for the changes have been added (for bug fixes / features) (if applicable)

Tested using the NuGet package from build artifacts in a new C# project

The official UWP docs will be updated after the package is live

The official code sample for My People shoulder tap will be updated after the package is live

What is the new behavior?

Adding new properties so you can send My People shoulder taps and include identifiers for telemetry.

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@andrewleader andrewleader requested a review from nmetulev January 22, 2018 17:24
@andrewleader
Copy link
Contributor Author

I'm currently testing this in actual projects with the generated Nuget packages

Andrew Bares added 2 commits January 22, 2018 10:31
FileNotFoundException was being thrown since System.Xml.ReaderWriter assembly couldn't be loaded. Same issue was fixed in the C#/C++ version previously, simply had to reference that package.
@andrewleader
Copy link
Contributor Author

This is ready for review, tested the new features in a new C# app using the NuGet package from build artifacts, all good!

@nmetulev nmetulev merged commit 135bebc into master Jan 23, 2018
@nmetulev nmetulev deleted the anbare/toast-additions-1801 branch January 23, 2018 03:20
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.

2 participants