search

Azure-Samples / ms-identity-java-webapp

A Java web application calling Microsoft graph that is secured using the Microsoft identity platform
MIT License
118 stars 105 forks source link

Minor edits and Permission changes #10

Closed ramya25 closed 5 years ago

ramya25 commented 5 years ago

Replacing delegated permission(User.ReadBasic.All) with basic permission(User.Read) and calling graph with /me endpoint.

navyasric commented 5 years ago

@ramya25 @sangonzal Can we merge this PR changes as soon as the Quickstart doc is merged?

jmprieur commented 5 years ago

@navyasric : why wait for the QS doc to be merged?

navyasric commented 5 years ago

@jmprieur I guess we could merge it independently. But the current quickstart that is live would not match these changes when the link to the repo or code is downloaded, for a short while till the quickstart doc PR is merged.

jmprieur commented 5 years ago

Got it, @navyasric : That makes sense

ramya25 commented 5 years ago

@ramya25 @sangonzal Can we merge this PR changes as soon as the Quickstart doc is merged?

@navyasric I made few changes reflecting the Quickstart doc. Please review and merge accordingly.

navyasric commented 5 years ago

LGTM 👍 Merging.