Skip to content

Added info about nItem parameter of SetItemState.#2512

Merged
colin-home merged 4 commits into
MicrosoftDocs:masterfrom
lihas:patch-1
Dec 17, 2020
Merged

Added info about nItem parameter of SetItemState.#2512
colin-home merged 4 commits into
MicrosoftDocs:masterfrom
lihas:patch-1

Conversation

@lihas
Copy link
Copy Markdown
Contributor

@lihas lihas commented Oct 3, 2020

@PRMerger10
Copy link
Copy Markdown
Contributor

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

Copy link
Copy Markdown
Collaborator

@TylerMSFT TylerMSFT left a comment

Choose a reason for hiding this comment

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

Thank you for adding this useful info.
Could we phrase this to be more direct: Pass -1 to apply the state change to all items.

@PRMerger9 PRMerger9 requested a review from TylerMSFT December 15, 2020 05:14
@PRMerger9
Copy link
Copy Markdown
Contributor

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

@lihas
Copy link
Copy Markdown
Contributor Author

lihas commented Dec 15, 2020

Thank you for adding this useful info.
Could we phrase this to be more direct: Pass -1 to apply the state change to all items.

Made the change as you suggested. The merge can proceed now.

Repeat edit to content.
@PRMerger18
Copy link
Copy Markdown
Contributor

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

Reword for context.
@PRMerger19
Copy link
Copy Markdown
Contributor

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

Copy link
Copy Markdown
Contributor

@colin-home colin-home left a comment

Choose a reason for hiding this comment

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

@lihas Thanks for the updates.

@colin-home colin-home merged commit c75fb8a into MicrosoftDocs:master Dec 17, 2020
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.

8 participants