-
Notifications
You must be signed in to change notification settings - Fork 456
[Bug] GravatarImageSource throws exception when offline #1816
Conversation
<Project Sdk="Microsoft.NET.Sdk"> | ||
|
||
<PropertyGroup> | ||
<TargetFrameworks>netcoreapp2.1;netcoreapp3.1;net461</TargetFrameworks> |
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.
fix build out of the box. netcoreapp2.1 is the end of support
} | ||
|
||
static void UpdateStatusBarAppearance(UIWindow window) | ||
static void UpdateStatusBarAppearance(UIWindow? window) |
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.
see line 97. window can be null
return Array.Empty<byte>(); | ||
|
||
return await response.Content.ReadAsByteArrayAsync(); | ||
return await response.Content.ReadAsByteArrayAsync(); |
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.
Can this return null
?
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.
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.
Looks good in general, just some observations
GravatarImageSource throws exception when offline
Fix Nullable
Description of Bug
Issues Fixed
Behavioral Changes
PR Checklist
approved