-
Notifications
You must be signed in to change notification settings - Fork 1.9k
[UWP] Fixed NRE load image from ImageSource #13135
Conversation
Hey @jsuarezruiz is it possible to add a UITest ? |
Sure, added!. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jsuarezruiz test is failing .
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hey @jsuarezruiz tests is failing on iOS and Android
@jsuarezruiz i comment your change and the test still passes, am i holding it wrong ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tested the nuget on the issue and it solves the issue. But that said the UITest added is not really testing this issue since Application is not null if we do it from the Xamarin.Forms side .
I also think this should target 4.8.0 , since is ra regression there
@jsuarezruiz tests is failing on UWP |
Description of Change
Fixed NRE load image from ImageSource in UWP.
Issues Resolved
API Changes
None
Platforms Affected
Behavioral/Visual Changes
None
Before/After Screenshots
Not applicable
Testing Procedure
Launch Core Gallery and navigate to the issue 13109. Without exceptions, the test has passed.
PR Checklist