Skip to content

Conversation

@ditman
Copy link
Contributor

@ditman ditman commented Dec 7, 2023

This PR:

  • Migrates the web implementation of the package to package:web. This allows this package to be used by apps that want to be compiled to WASM.
  • Updates the example app a little bit
  • Fixes the platform_channel test so it does what it seems to want to do :)

This is what the example web app looks like now:

Screenshot 2023-12-06 at 10 27 29 PM

Thanks for this package @codebird!

Comment on lines -7 to +8
sdk: '>=2.12.1 <3.0.0'
flutter: ">=2.5.0"
sdk: ^3.2.0
flutter: ^3.16.0
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hm, the android version doesn't seem to like this flutter update...

$ flutter run -d emu

Launching lib/main.dart on Android SDK built for x86 in debug mode...
package identifier or launch activity not found.
Please check /work/ditman/codebird-flutter_app_icons/example/android/app/src/main/AndroidManifest.xml for errors.
No application found for TargetPlatform.android_x86.
Is your project missing an android/app/src/main/AndroidManifest.xml?
Consider running "flutter create ." to create one.

Would you be willing to update this package to a more recent version of Flutter @codebird?

Copy link
Owner

Choose a reason for hiding this comment

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

Hello @ditman,

Thanks for the PR.

Will check it out, and see if I can fix on android.

Yes I surely would like to keep it working with the most recent version of flutter, and keep it going... Time is always the issue.

Copy link
Owner

Choose a reason for hiding this comment

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

The issue is with DONT_KILL_APP flag, android doesn't like it anymore. Going to merge your pull request as it enhances the package... Will try to figure something out for Android.

@codebird codebird merged commit a61dedb into codebird:master Dec 7, 2023
@ditman ditman deleted the migrate-to-pkg-web branch December 7, 2023 20:24
@ditman
Copy link
Contributor Author

ditman commented Dec 7, 2023

Thanks for the super fast merge! 🙏

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