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

Add explanation for serializing a DataMember with EmitDefaultValue as false. #7019

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

LI-Yanting
Copy link

Summary

Explain explicitly that a DataMember with a non default value will still be serialized despite its EmitDefaultValue = false.

@LI-Yanting LI-Yanting requested a review from a team as a code owner August 9, 2021 08:10
@dotnet-issue-labeler
Copy link

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@opbld30
Copy link

opbld30 commented Aug 9, 2021

Docs Build status updates of commit 243873f:

✅ Validation status: passed

File Status Preview URL Details
samples/snippets/csharp/VS_Snippets_CFX/datamemberattribute/cs/overview.cs ✅Succeeded View

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@adamsitnik
Copy link
Member

cc @StephenMolloy @HongGit

@BillWagner
Copy link
Member

ping @carlossanlop to route for review.

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.

4 participants