Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

No need to show SMS sent message for non-loggedin users #28261

Closed
ajiragroup opened this issue Oct 23, 2024 · 2 comments · Fixed by #28278
Closed

No need to show SMS sent message for non-loggedin users #28261

ajiragroup opened this issue Oct 23, 2024 · 2 comments · Fixed by #28278
Labels
bug released

Comments

@ajiragroup
Copy link
Contributor

I have set up sms notification when applicant submits an application. Then this message is shown to user:

image

This is actually not good. The users do not need to know what numbers have been sent sms to. Some times, i might set up an sms alert for me to notify when someone submits an sms. That time, the public user will know who received an sms.
@ajiragroup ajiragroup added the bug label Oct 23, 2024
@sumitbhanushali sumitbhanushali mentioned this issue Oct 24, 2024
Merged
This was referenced Oct 24, 2024
Merged
Merged
@frappe-pr-bot
Copy link
Collaborator

🎉 This issue has been resolved in version 15.46.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@frappe-pr-bot
Copy link
Collaborator

🎉 This issue has been resolved in version 14.84.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 13, 2024
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Assignees
No one assigned
Labels
bug released
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: dont display phone numbers to which sms was sent to frappe/frappe
2 participants
@frappe-pr-bot @ajiragroup