Skip to content

Specify data type matching#6764

Merged
ktoliver merged 2 commits intoMicrosoftDocs:livefrom
way0utwest:patch-34
Nov 23, 2021
Merged

Specify data type matching#6764
ktoliver merged 2 commits intoMicrosoftDocs:livefrom
way0utwest:patch-34

Conversation

@way0utwest
Copy link
Copy Markdown
Contributor

Since this procedure requires data types of values to match the definition, added notes. Specifically, the rule must be nvarchar. A varchar relying on implicit conversion will not work.

Since this procedure requires data types of values to match the definition, added notes. Specifically, the rule must be nvarchar. A varchar relying on implicit conversion will not work.
@PRMerger20
Copy link
Copy Markdown
Contributor

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

@VanMSFT
Copy link
Copy Markdown
Member

VanMSFT commented Aug 31, 2021

Thanks @way0utwest! We really appreciate the help!

@rohitnayakmsft - Hey Rohit, can you please verify this?

@ktoliver ktoliver added the aq-pr-triaged tracking label for the PR review team label Aug 31, 2021
@ktoliver
Copy link
Copy Markdown
Contributor

@rohitnayakmsft Could you review this pull request? Thanks!

@rohitnayakmsft
Copy link
Copy Markdown
Contributor

#sign-off

@PRMerger12
Copy link
Copy Markdown
Contributor

@rohitnayakmsft: I'm sorry - only the author of this article, @VanMSFT, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@PRMerger15
Copy link
Copy Markdown
Contributor

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

@ktoliver ktoliver merged commit 5f4122d into MicrosoftDocs:live Nov 23, 2021
@ktoliver
Copy link
Copy Markdown
Contributor

Thanks @rohitnayakmsft.

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