MaikuB / flutter_local_notifications

A Flutter plugin for displaying local notifications on Android, iOS, macOS and Linux
2.47k stars 1.4k forks source link

[flutter_local_notifications_linux] Changed conditional import (required for WASM compilation) #2362

Closed AlexBacich closed 4 months ago

MaikuB commented 4 months ago

Do you have more context on what this was supposed to solve? Is it do with https://github.com/dart-lang/sdk/issues/55948? If so, my understanding was to the 3.22 release where WASM was started being considered stable that the fix would be to update the Flutter SDK

AlexBacich commented 4 months ago

@MaikuB , thanks for your input. With current Channel stable, 3.22.2 it is not working. But yes - eventually the fix will be in future releases. https://github.com/flutter/flutter/issues/149984

MaikuB commented 4 months ago

Thanks for the reply. Going to close this in favour of the tooling being fixed. The comments I've seen would indicate this was more a problem with the conditional import not working as intended and returning the wrong response. Furthermore, I've seen more info on using ` as a way to detect FFI if is supported. It's possible I've missed it but I've not seen info that indicates checking fordart.library.iois another way to confirm if a platform supports FFI. If anything, I've seen more to indicatedart.library.iois more generic and covers mobile use cases as well. The first-party plugins still make use ofdart.library.ffi`