Skip to content
This repository was archived by the owner on Oct 29, 2024. It is now read-only.

Added uwp sample for Todolist#62

Open
AutumnEvans418 wants to merge 2 commits intodotnet:mainfrom
AutumnEvans418:addedUwpSample
Open

Added uwp sample for Todolist#62
AutumnEvans418 wants to merge 2 commits intodotnet:mainfrom
AutumnEvans418:addedUwpSample

Conversation

@AutumnEvans418
Copy link
Copy Markdown

@AutumnEvans418 AutumnEvans418 commented Jan 24, 2020

Added a UWP sample for the todo sample:

image

The event log logger provider in Microsoft extensions is not implemented for UWP and will throw an exception if you run the app, so I updated the app to remove that logger:

  .ConfigureLogging(d =>
                {
                    d.ClearProviders();
                    d.AddConsole();
                })

This was the only change I had to make for UWP to work.

@dnfclas
Copy link
Copy Markdown

dnfclas commented Jan 24, 2020

CLA assistant check
All CLA requirements met.

@AutumnEvans418 AutumnEvans418 requested a review from Eilon January 24, 2020 03:38
@Eilon
Copy link
Copy Markdown
Contributor

Eilon commented Jan 24, 2020

Very cool, I will take a look at this!

Base automatically changed from master to main March 16, 2021 18:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants