Skip to content

feat: Support for Windows.Data.Xml.Dom APIs #3070

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

Merged
merged 17 commits into from
May 9, 2020

Conversation

MartinZikmund
Copy link
Member

@MartinZikmund MartinZikmund commented Apr 27, 2020

GitHub Issue (If applicable): closes #3069

PR Type

What kind of change does this PR introduce?

  • Feature

What is the current behavior?

APIs not supported.

What is the new behavior?

All APIs which can be forwarded to System.Xml are implemented.

PR Checklist

Please check if your PR fulfills the following requirements:

@gitpod-io
Copy link

gitpod-io bot commented Apr 27, 2020

@MartinZikmund
Copy link
Member Author

Will add some simple unit tests.

@MartinZikmund MartinZikmund marked this pull request as ready for review April 28, 2020 06:15
@MartinZikmund
Copy link
Member Author

@jeromelaban Should I create a new section in PackageDiffIgnore? What would be the base version for it?

@jeromelaban
Copy link
Member

jeromelaban commented Apr 28, 2020

Yes you can. Use a section similar to this one:

https://github.com/unoplatform/uno/pull/3044/files#diff-49d44d5b2511b8a0bd889bc14a811169R809

GitHub
Feature First implementation of WinUI 2.4's RadialGradientBrush PR Checklist Please check if your PR fulfills the following requirements:

Docs have been added/updated which fit documentation ...

@jeromelaban
Copy link
Member

Thanks Martin!

@pkar70
Copy link
Contributor

pkar70 commented May 4, 2020

I will use it in my Live Tile implementation :)

@MartinZikmund MartinZikmund added the ready-to-merge Automatically merge the PR once all '.mergify.yml' policies are met label May 6, 2020
@agneszitte
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@unoplatform unoplatform deleted a comment from MartinZikmund May 8, 2020
private string GetTestXml()
{
using var stream = typeof(Given_XmlDocument).Assembly.GetManifestResourceStream("Uno.UI.Tests.Windows_Data_Xml.DomTests.basictest.xml");
using var streamReader = new StreamReader(stream);
Copy link
Contributor

Choose a reason for hiding this comment

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

@mergify mergify bot merged commit 17916d1 into unoplatform:master May 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge Automatically merge the PR once all '.mergify.yml' policies are met
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support for Windows.Data.Xml.Dom
6 participants