Skip to content

Feature: Create and manage aad application#1014

Merged
timotheeguerin merged 47 commits intomasterfrom
feature/sdk-service-principal
Jan 22, 2018
Merged

Feature: Create and manage aad application#1014
timotheeguerin merged 47 commits intomasterfrom
feature/sdk-service-principal

Conversation

@timotheeguerin
Copy link
Member

@timotheeguerin timotheeguerin commented Jan 19, 2018

fix #996
image

image

@codecov
Copy link

codecov bot commented Jan 22, 2018

Codecov Report

❗ No coverage uploaded for pull request base (master@75df152). Click here to learn what that means.
The diff coverage is 37.1%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1014   +/-   ##
=========================================
  Coverage          ?   53.87%           
=========================================
  Files             ?      816           
  Lines             ?    19559           
  Branches          ?     2195           
=========================================
  Hits              ?    10537           
  Misses            ?     9022           
  Partials          ?        0
Impacted Files Coverage Δ
...tails/programatic-usage/programing-sample/index.ts 100% <ø> (ø)
src/common/constants/constants.ts 100% <ø> (ø)
app/components/account/details/index.ts 100% <ø> (ø)
src/common/utils/secure-utils.ts 81.81% <0%> (ø)
app/models/server-error.ts 55.78% <0%> (ø)
...pp/services/ms-graph/core/aad-graph-list-getter.ts 10.34% <10.34%> (ø)
app/services/ms-graph/core/index.ts 100% <100%> (ø)
...age/aad-credentials-picker/aad-app-picker/index.ts 100% <100%> (ø)
...aad-credentials-picker/create-new-aad-app/index.ts 100% <100%> (ø)
...ponents/account/details/programatic-usage/index.ts 100% <100%> (ø)
... and 44 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 75df152...ee89023. Read the comment docs.

@timotheeguerin timotheeguerin merged commit f1c22de into master Jan 22, 2018
@timotheeguerin timotheeguerin deleted the feature/sdk-service-principal branch January 22, 2018 22:37
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.

Add connection with service principal

1 participant