Skip to content

Conversation

@stuartmorgan-g
Copy link
Contributor

This adds a Windows section to the existing iOS and Android section, following the same structure as those examples, and using the code from flutter/flutter#106754

Part of flutter/flutter#79204

Presubmit checklist

This adds a Windows section to the existing iOS and Android section, following the same structure as those examples, and using the code from flutter/flutter#106754

Part of flutter/flutter#79204
if device.batteryState == UIDevice.BatteryState.unknown {
result(FlutterError(code: "UNAVAILABLE",
message: "Battery info unavailable",
message: "Battery level not available.",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is an opportunistic fix; there was a pointless minor inconsistency between the Android and iOS examples here, so I unified them to use the same message.

Copy link
Contributor

@sfshaza2 sfshaza2 left a comment

Choose a reason for hiding this comment

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

lgtm. Thanks again, @stuartmorgan!

@sfshaza2 sfshaza2 merged commit 2ad2c1f into main Jun 30, 2022
@sfshaza2 sfshaza2 deleted the platform-channels-windows-example branch June 30, 2022 23:41
stuartmorgan-g added a commit that referenced this pull request Jan 20, 2023
This adds a Linux section to the example implementation instructions, following the same structure as the existing examples, and using the code from flutter/flutter#108068

This is the Linux equivalent of #7328

Part of flutter/flutter#79204
sfshaza2 added a commit that referenced this pull request Jan 26, 2023
* Add Linux to platform channel example

This adds a Linux section to the example implementation instructions, following the same structure as the existing examples, and using the code from flutter/flutter#108068

This is the Linux equivalent of #7328

Part of flutter/flutter#79204

* Update src/development/platform-integration/platform-channels.md

Co-authored-by: Loïc Sharma <737941+loic-sharma@users.noreply.github.com>

* Update src/development/platform-integration/platform-channels.md

Co-authored-by: Loïc Sharma <737941+loic-sharma@users.noreply.github.com>

* Update src/development/platform-integration/platform-channels.md

Co-authored-by: Shams Zakhour (ignore Sfshaza) <44418985+sfshaza2@users.noreply.github.com>

* Update src/development/platform-integration/platform-channels.md

Co-authored-by: Shams Zakhour (ignore Sfshaza) <44418985+sfshaza2@users.noreply.github.com>

* Update src/development/platform-integration/platform-channels.md

Co-authored-by: Shams Zakhour (ignore Sfshaza) <44418985+sfshaza2@users.noreply.github.com>

Co-authored-by: Loïc Sharma <737941+loic-sharma@users.noreply.github.com>
Co-authored-by: Parker Lougheed <parlough@gmail.com>
Co-authored-by: Shams Zakhour (ignore Sfshaza) <44418985+sfshaza2@users.noreply.github.com>
stuartmorgan-g added a commit that referenced this pull request Apr 5, 2023
This adds a macOS section to the example implementation instructions, following the same structure as the existing examples, and using the code from flutter/flutter#123141

This is the macOS equivalent of #7328 and #8127

Fixes flutter/flutter#79204
sfshaza2 pushed a commit that referenced this pull request Apr 5, 2023
This adds a macOS section to the example implementation instructions,
following the same structure as the existing examples, and using the
code from flutter/flutter#123141

This is the macOS equivalent of
#7328 and
#8127

Fixes flutter/flutter#79204

## Presubmit checklist
- [x] This PR doesn’t contain automatically generated corrections
(Grammarly or similar).
- [x] This PR follows the [Google Developer Documentation Style
Guidelines](https://developers.google.com/style) — for example, it
doesn’t use _i.e._ or _e.g._, and it avoids _I_ and _we_ (first person).
- [x] This PR uses [semantic line
breaks](https://github.com/dart-lang/site-shared/blob/master/doc/writing-for-dart-and-flutter-websites.md#semantic-line-breaks)
of 80 characters or fewer.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants