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

Added graph controls #2055

Merged
merged 209 commits into from
May 23, 2018
Merged

Added graph controls #2055

merged 209 commits into from
May 23, 2018

Conversation

OGcanviz
Copy link
Contributor

@OGcanviz OGcanviz commented May 3, 2018

Graph UWP Controls provide a robust foundation for controls that can be integrated into UWP applications. The goal behind these controls is to make it easier for developers to harness the power of the Graph in their applications, providing valuable functionality like file support and tasks.

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)
  • Header has been added to all new source files (run build/UpdateHeaders.bat)
  • Contains NO breaking changes

Breaking change

MicrosoftGraphService IsConnected (protected property) is renamed to IsAuthenticated and it's public

@dnfclas
Copy link

dnfclas commented May 3, 2018

CLA assistant check
All CLA requirements met.

@Vijay-Nirmal
Copy link
Contributor

Can you add some PR description? It will be very helpful

@OGcanviz
Copy link
Contributor Author

OGcanviz commented May 23, 2018

@nmetulev @michael-hawker @azchohfi Thank your all for the further advice and help, your proposed changes are all done (except a few of them which were already replied separately), could you review if they are OK for merging now? Thanks again.

@azchohfi azchohfi merged commit 0f80544 into CommunityToolkit:master May 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.