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.

Sign up for GitHub

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

Jump to bottom

Fix namespace issue (.AppConfig) Add .Bind() info #33374

Merged
merged 1 commit into from
Jun 17, 2019
Merged

Fix namespace issue (.AppConfig) Add .Bind() info #33374

merged 1 commit into from
Jun 17, 2019

Conversation

jennyf19
Copy link
Contributor

@PRMerger20 PRMerger20 requested a review from rwike77 June 14, 2019 22:59
@PRMerger20
Copy link
Contributor

@jennyf19 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

rwike77
rwike77 approved these changes Jun 14, 2019
View reviewed changes
Copy link
Contributor

@rwike77 rwike77 left a comment

Choose a reason for hiding this comment

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

Looks good to me, thanks.

@rwike77
Copy link
Contributor

rwike77 commented Jun 14, 2019

#sign-off

@chbain chbain merged commit 8a93894 into MicrosoftDocs:master Jun 17, 2019
@jennyf19 jennyf19 deleted the jennyf/docFix32367 branch June 17, 2019 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rwike77 rwike77 rwike77 approved these changes

Assignees

@rwike77 rwike77

Labels
active-directory/svc Change sent to author develop/subsvc ready-to-merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@jennyf19 @PRMerger20 @rwike77 @PRMerger8 @chbain