-
Notifications
You must be signed in to change notification settings - Fork 9.7k
[ios_platform_images] Fix crash when parameter extension is null #2654
[ios_platform_images] Fix crash when parameter extension is null #2654
Conversation
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.
Thanks, looks good after the version number change.
@@ -1,3 +1,7 @@ | |||
## 0.1.0+3 |
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.
Please change this to 0.1.1. The "+N" nomenclature is for changes that don't affect the binary.
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 made the change for you so we can get this landed.
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.
@gaaclarke 👍 Seems other plugin causes CI red.
/// By default `gestureNavigationEnabled` is false. | ||
final bool gestureNavigationEnabled; | ||
|
||
/// The value used for the HTTP User-Agent: request header. |
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 don't know wether we need to open another PR to merge this, I think it's trivial. If you think we need, I can revert this. :)
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'm fine with it =)
…tter#2654) Co-authored-by: Aaron Clarke <aaclarke@google.com>
…tter#2654) Co-authored-by: Aaron Clarke <aaclarke@google.com>
…tter#2654) Co-authored-by: Aaron Clarke <aaclarke@google.com>
Description
Crash when
extension
is null.Related Issues
N/A.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes (
[x]
). This will ensure a smooth and quick review process.///
).flutter analyze
) does not report any problems on my PR.Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?