Skip to content

move sms receive code path to a worker model to avoid anr on message receive#391

Merged
octoshrimpy merged 2 commits intoquik-sms:masterfrom
gavine99:anr-when-receiving-a-text-380
Mar 28, 2025
Merged

move sms receive code path to a worker model to avoid anr on message receive#391
octoshrimpy merged 2 commits intoquik-sms:masterfrom
gavine99:anr-when-receiving-a-text-380

Conversation

@gavine99
Copy link
Copy Markdown
Contributor

@gavine99 gavine99 commented Mar 28, 2025

implement new receivesms worker.
modify sms receive interactor to be used by new worker and more logging.
create custom workfactory that supports injection into workers.
modify housekeeping worker to use injection.

possibly closes #380

possibly closes #384

gavine99 added 2 commits March 28, 2025 16:27
modify sms receive interactor to be used by new worker and add more logging.
create custom workfactory that supports injection into workers.
modify housekeeping worker to use injection.
@octoshrimpy octoshrimpy merged commit 8a3ca00 into quik-sms:master Mar 28, 2025
2 checks passed
@gavine99
Copy link
Copy Markdown
Contributor Author

is there a phrase i can use on commit comments to notify of a possible close that won't auto close on merge?

@octoshrimpy
Copy link
Copy Markdown
Collaborator

I believe as long as the issue mention has any other word before it other than "closes"?

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.

Incoming message is not displayed in my inbox immediately 🐞 [BUG] ANR when receiving a text

2 participants