Skip to content

Enable Android conversation features#434

Merged
octoshrimpy merged 5 commits intoquik-sms:masterfrom
Choucroute-melba:feat/android-conversation
Apr 23, 2025
Merged

Enable Android conversation features#434
octoshrimpy merged 5 commits intoquik-sms:masterfrom
Choucroute-melba:feat/android-conversation

Conversation

@Choucroute-melba
Copy link
Copy Markdown
Contributor

Hi, this is for #423

This PR add the necessary shortcuts to the system and to notifications so Android can enable it's conversation features for QUIK : notifications will be displayed on top, you can define priority convs and you can uses and android's default conversations widget.

I also added a Recipient.getThemedIcon function to get unified icons between notifications, launcher shortcuts and in app view.

This is also a first step for bubbles support, but some more work is needed.

Screenshot_20250421-002905_Trebuchet
Screenshot_20250421-003116_QUIK-Debug
Screenshot_20250421-005756_Trebuchet
Screenshot_20250421-005010_Trebuchet

@octoshrimpy
Copy link
Copy Markdown
Collaborator

Thank you so much! going over the code now

@octoshrimpy octoshrimpy merged commit 6263e7b into quik-sms:master Apr 23, 2025
2 checks passed
@gavine99
Copy link
Copy Markdown
Contributor

@octoshrimpy fyi, i had to revert the commits in this pr from my local build because it broke notifications for group mms. not sure about single recipient mms or sms - i haven't tested for those.

@octoshrimpy
Copy link
Copy Markdown
Collaborator

@octoshrimpy fyi, i had to revert the commits in this pr from my local build because it broke notifications for group mms. not sure about single recipient mms or sms - i haven't tested for those.

that's no good, I didn't have any problems when testing. I wonder what's up

@gavine99
Copy link
Copy Markdown
Contributor

a had a quick browse of the logcat and it seemed it was maybe feeding a null image to the notification when a new message arrived. the group mms had 1 recipient with an image associated, and 3 recipients with no image associated.

@Choucroute-melba
Copy link
Copy Markdown
Contributor Author

Hi, I made a fix in #445 that I tested in shortcuts but I couldn't test notifications so tell me if there is anything else I can do

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.

3 participants