Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: setOnBackgroundMessage() is now a Future<void> that will await for the function to complete. #499

Merged
merged 7 commits into from
Dec 13, 2023

Conversation

ttypic
Copy link
Contributor

@ttypic ttypic commented Dec 6, 2023

Copied #498

I made the call to the onPushBackgroundMessage await for the handler to complete.
I changed handleBackgroundMessage() to become a Future<void> and async/await.
I made the BackgroundMessageHandler a Future<void> instead of void so that we can async/await for it to be completed.
handleBackgroundMessage now handles when _onBackgroundMessage is defined as void or Future<void>  It checks to see if it's a Future and then awaits it if it is.
Changed the cast on BackgroundMessageHandler to dynamic, this allows it to be defined as both void and Future<void>
@github-actions github-actions bot temporarily deployed to staging/pull/499/features December 6, 2023 16:14 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/499/dartdoc December 6, 2023 16:15 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/499/features December 6, 2023 18:47 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/499/dartdoc December 6, 2023 18:47 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/499/features December 11, 2023 23:23 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/499/dartdoc December 11, 2023 23:23 Inactive
@ttypic ttypic marked this pull request as ready for review December 13, 2023 18:56
@ttypic ttypic changed the title Test run ci to check all tests are passed feat: setOnBackgroundMessage() is now a Future<void> that will await for the function to complete. Dec 13, 2023
@sacOO7 sacOO7 self-requested a review December 13, 2023 21:24
Copy link
Collaborator

@sacOO7 sacOO7 left a comment

Choose a reason for hiding this comment

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

LGTM

@ttypic ttypic merged commit 39bc618 into main Dec 13, 2023
8 checks passed
@ttypic ttypic deleted the test-run-ci-for-forked-pr branch December 13, 2023 22:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants