Skip to content

Conversation

@RogerDX9
Copy link
Contributor

Fixed document descritption and return value

Fixed document descritption and return value
@PRMerger16
Copy link

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

Copy link
Member

@stevewhims stevewhims left a comment

Choose a reason for hiding this comment

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

I have a mail out to the feature team to double-check that the descriptions here are accurate, so please stand by. The "(function name contains a typo, should be GetMemberName)" part looks like product feedback, so we can't merge that into the content. Once I have all the info, I'll make a new commit to this PR and then approve and merge it. Thanks!

Copy link
Member

@stevewhims stevewhims left a comment

Choose a reason for hiding this comment

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

With input from the feature team, I did an edit pass on the PR.

@msftclas
Copy link

msftclas commented Jan 31, 2020

CLA assistant check
All CLA requirements met.

@stevewhims
Copy link
Member

stevewhims commented Jan 31, 2020

Hi, @spoiltos, I'm ready to merge and complete this PR, it's just currently blocked on your signing the license/cla. Please see the check item in the UI below. Thanks!

@PRMerger9
Copy link

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

@PRMerger9 PRMerger9 requested a review from stevewhims January 31, 2020 21:16
@stevewhims stevewhims merged commit 8d9d910 into MicrosoftDocs:docs Jan 31, 2020
@RogerDX9
Copy link
Contributor Author

Hi @stevewhims, I've signed the license/cla. Also I like your new changes, thanks a lot!

@RogerDX9 RogerDX9 deleted the patch-1 branch January 31, 2020 21:27
@stevewhims
Copy link
Member

Thank you, @spoiltos! :)

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.

5 participants