Skip to content

Conversation

gavine99
Copy link
Contributor

fix no creation of a new conversation when no existing conversation to member of a multi-recipient sms not sent as group.

steps to reproduce existing bug;

  1. compose new message to no recipients using floating '+' button on main activity
  2. add a recipient with existing message(s) in their conversation
  3. then also add a recipient with no existing messages/conversation
  4. enter a message body
  5. send >
  6. check sent conversations list in main activity;
  7. recipient with existing conversation will have sent message in their conversation
  8. recipient with no existing conversation will not have a new conversation created with the new message

encountered this bug whilst testing for pr #344

gavine99 and others added 5 commits March 12, 2025 14:14
…activity) when sending from a new convo as group. also fix associated bug that was sending sms to individual recipients in 'send as group mode' when send as group pref was saved as true
…o member of a multi-recipient sms not sent as group
…end-320-haserror-sendasgroup-fix

fix bug causing compose activity to finish() when it wasn't supposed to
…k-fix

Remove `DependencyInfoBlock` in APK
@octoshrimpy
Copy link
Owner

conflicts, otherwise looks good. :)

gavine99 added 2 commits March 13, 2025 10:08
…o member of a multi-recipient sms not sent as group
…'t-start-new-conversations' into send-to-multiple-addresses-doesn't-start-new-conversations

# Conflicts:
#	presentation/src/main/java/com/moez/QKSMS/feature/compose/ComposeViewModel.kt
@gavine99
Copy link
Contributor Author

rebased pr to latest master and should be ready to pull

@octoshrimpy octoshrimpy merged commit 1aee6f0 into octoshrimpy:master Mar 15, 2025
2 checks passed
@gavine99 gavine99 deleted the send-to-multiple-addresses-doesn't-start-new-conversations branch March 17, 2025 02:59
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