Skip to content

Conversation

TejaswaniRai
Copy link

Which issue does this PR close?

Closes #60 - Title: Voice-to-SMS Input Feature

Rationale for this change

Currently, Spamlyser Pro only supports manual text input.
Adding voice-to-SMS input improves accessibility, usability, and makes the platform feel more modern and interactive.

What changes are included in this PR?

  • Added microphone button inside SMS input field.
  • Auto-fills transcribed text into the SMS input box.
  • Error handling + compatibility note for browsers.

Are these changes tested?

image ✅ Tested locally on Chrome (Windows) with microphone input. ✅ Verified transcription auto-fills SMS box.

Copy link

github-actions bot commented Sep 8, 2025

Thanks for creating a PR for your Issue! ☺️

We'll review it as soon as possible.
In the meantime, please double-check the file changes and ensure that all commits are accurate.

If there are any unresolved review comments, feel free to resolve them. 🙌🏼

@eccentriccoder01
Copy link
Owner

Great work @TejaswaniRai! Sorry for the late response, I was caught up with a lot of things... Could you please provide a working streamlit cloud deployment link of your updated fork/PR? Thanks!

@shaanlabs
Copy link
Collaborator

shaanlabs commented Sep 13, 2025

@TejaswaniRai This branch has conflicts that must be resolved

@eccentriccoder01
Copy link
Owner

@shaanlabs kindly don't merge the PR into the master unless @TejaswaniRai provides a Streamlit Cloud deployment link.

@TejaswaniRai
Copy link
Author

Hi, @eccentriccoder01 , I’m really sorry, but I won’t be able to continue with this issue/PR. I’ve been sick and busy with my academic exams , and I haven’t been able to give this the attention it deserves.

I know it’s unprofessional to leave midway and I truly regret not being able to follow through. To avoid blocking progress, I’m unassigning myself so that someone else can take this forward.

Thank you for understanding, and apologies once again for the inconvenience caused.

@eccentriccoder01
Copy link
Owner

eccentriccoder01 commented Sep 27, 2025

You’ve already put in significant effort on this @TejaswaniRai, and I really appreciate it

So I’m re-assigning this back to you...but don’t worry, you don’t have to spend more time on it. I’ll take care of running, deploying, and fixing any issues myself whenever I get time, and then merge it so you still get the points you absolutely deserve

Please take care of your health (these things happen, no stress at all), and wishing you all the best for your exams🌸
You’ve already done great, don't worry!

@shaanlabs
Copy link
Collaborator

@eccentriccoder01 I will not merge any pr

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.

Title: Voice-to-SMS Input Feature
3 participants