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

Send message from call log/history removes the + from the front of the number => sms ca'n be sent #294

Closed
7 tasks done
bulletproof-coding opened this issue Jan 27, 2025 · 4 comments · Fixed by #301
Closed
7 tasks done
Assignees
Labels
bug Something is not working

Comments

@bulletproof-coding
Copy link

Checklist

  • I can reproduce the bug with the latest version given here.
  • I made sure that there are no existing issues - open or closed - to which I could contribute my information.
  • I made sure that there are no existing discussions - open or closed - to which I could contribute my information.
  • I have read the FAQs inside the app (Menu -> About -> FAQs) and my problem isn't listed.
  • I have taken the time to fill in all the required details. I understand that the bug report will be dismissed otherwise.
  • This issue contains only one bug.
  • I have read and understood the contribution guidelines.

Affected app version

1.1.4

Affected Android/Custom ROM version

Android 12

Affected device model

Volla Phone X

How did you install the app?

F-Droid / IzzyOnDroid

Steps to reproduce the bug

Go to call log
Click on 3 dots near a contact or raw number -> Send sms
The sms opens and the number does not have the + in front resulting in impossibility to send the message

Expected behavior

The number should be pasted exactly as it is with + in front so the sms can be sent.

Actual behavior

Carrier refuses to send message and number can't be edited.

Screenshots/Screen recordings

No response

Additional information

No response

@bulletproof-coding bulletproof-coding added bug Something is not working needs triage Issue is not yet ready for PR authors to take up labels Jan 27, 2025
@Aga-C Aga-C removed the needs triage Issue is not yet ready for PR authors to take up label Jan 27, 2025
@jayb-g
Copy link

jayb-g commented Jan 28, 2025

Experiencing same. This probably started happening around after recent updates of fossify messages.

Expected behavior

The number should be pasted exactly as it is with + in front so the sms can be sent.

It used to work fine before.

It still works fine if you use stock SMS app or qksms.

@naveensingh naveensingh self-assigned this Jan 28, 2025
@JimmyS83
Copy link

JimmyS83 commented Jan 28, 2025

Confirming this.

Is it possible keep older release as well here on github? This is exactly case when it could be usefull... Testing of possible new bug and for temporary reasons until fix is available...

EDIT: Apologies, mea culpa! I must been overworked today misclicking Releases hyperlink.

@naveensingh
Copy link
Member

All releases are on Github: https://github.com/FossifyOrg/Messages/releases

@naveensingh
Copy link
Member

If anyone would like to test out a fix, here's a debug APK: https://github.com/FossifyOrg/Messages/actions/runs/13089390827/artifacts/2521307580

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is not working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants