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.

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

Enhancing the SqlDbType with Json #103985

Merged
merged 5 commits into from
Jul 11, 2024
Merged

Enhancing the SqlDbType with Json #103985

merged 5 commits into from
Jul 11, 2024

Conversation

saurabh500
Copy link
Contributor

@saurabh500 saurabh500 commented Jun 25, 2024

Fixes #103925

The API review is approved.

The docs changes will need to be made in https://github.com/dotnet/dotnet-api-docs/blob/main/xml/System.Data/SqlDbType.xml and can be separated from this PR.

@dotnet-policy-service dotnet-policy-service bot added the community-contribution Indicates that the PR has been added by a community member label Jun 25, 2024
Copy link
Contributor

Tagging subscribers to this area: @roji, @ajcvickers
See info in area-owners.md if you want to be subscribed.

@jkoritzinsky
Copy link
Member

You'll need to add this member to the enum in the corresponding ref assembly System.Data.Common/ref/System.Data.Common.cs as well.

@saurabh500
Copy link
Contributor Author

Thanks @jkoritzinsky I will update the ref assembly. What about any docs related changes?

@jkoritzinsky
Copy link
Member

For docs, I believe you can put XML comments on the change in the src/ folder and they should propagate to docs when the next version with your change ships.

@carlossanlop can confirm.

@saurabh500
Copy link
Contributor Author

@roji this is ready for review. I have answered the question about the docs in the PR description.

@saurabh500
Copy link
Contributor Author

cc @apoorvdeshmukh @deepaksa1 for your awareness

@roji roji merged commit 2fe4afa into dotnet:main Jul 11, 2024
8 of 12 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Aug 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Expose Json in SqlDbType
4 participants