Skip to content

Fix escaped XML characters #8565

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.

Sign up for GitHub

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

Jump to bottom
Merged
gewarren merged 1 commit into from
Oct 24, 2022
Merged

Fix escaped XML characters #8565

gewarren merged 1 commit into from
Oct 24, 2022

Conversation

@Kissaki
Copy link
Contributor

@Kissaki Kissaki commented Oct 23, 2022

Fixes #7776

@Kissaki Kissaki requested a review from a team as a code owner October 23, 2022 14:35
@ghost ghost added community-contribution Indicates that the PR has been added by a community member area-System.Xml labels Oct 23, 2022
@ghost
Copy link

ghost commented Oct 23, 2022

Tagging subscribers to this area: @dotnet/area-system-xml
See info in area-owners.md if you want to be subscribed.

Issue Details

Fixes #7776

Author: Kissaki
Assignees: -
Labels:

area-System.Xml, community-contribution
Milestone: -

@opbld34
Copy link

opbld34 commented Oct 23, 2022

Learn Build status updates of commit 8fc853f:

✅ Validation status: passed

File Status Preview URL Details
snippets/csharp/System.Xml/XmlElement/InnerText/source.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:

gewarren
gewarren approved these changes Oct 24, 2022
View reviewed changes
Copy link
Contributor

@gewarren gewarren left a comment

Choose a reason for hiding this comment

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

Thanks @Kissaki. I checked the preview and it now matches the actual output you see when you run the program.

@gewarren gewarren merged commit 48929ea into dotnet:main Oct 24, 2022
@Kissaki Kissaki deleted the xml-escape branch June 4, 2024 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gewarren gewarren gewarren approved these changes

Assignees

No one assigned

Labels

area-System.Xml community-contribution Indicates that the PR has been added by a community member

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Incorrect output for XmlElement.InnerText property in example code

3 participants

@Kissaki @opbld34 @gewarren