Skip to content

add voice message#365

Open
sarahelsagheir wants to merge 3 commits into
munafio:masterfrom
sarahelsagheir:voice-message
Open

add voice message#365
sarahelsagheir wants to merge 3 commits into
munafio:masterfrom
sarahelsagheir:voice-message

Conversation

@sarahelsagheir

Copy link
Copy Markdown

No description provided.

@munafio

munafio commented Sep 12, 2024

Copy link
Copy Markdown
Owner

@sarahelsagheir can you please provide details, demo ?

@sarahelsagheir

Copy link
Copy Markdown
Author

@sarahelsagheir can you please provide details, demo ?

Add Voice Message Feature Similar to Messenger

Description: This update introduces a voice message feature similar to Messenger, with a default recording time set to 1 minute. Users can customize this duration and the file upload size.

How to Customize:

[ ] - Recording Duration: Modify const MAX_RECORDING_TIME = 60; in the VoiceMessage.js file.
[ ] - File Upload Size: Adjust settings in the chatify.php file.

first:
second

Demo
https://github.com/user-attachments/assets/f3e3cc31-3410-49f0-9369-04a3c0b45286

@monzer15

Copy link
Copy Markdown

Waiting for this great PR

@Makbari6143

Copy link
Copy Markdown

Waiting for this

@SlugVortex

Copy link
Copy Markdown

Deployment Risk Assessment 🚦

Risk Score: 10/100 ✅ Low Risk

Blast Radius: Limited scope of affected services with no historical incidents or risk indicators.

Key Factors:

  • Low risk score indicating minimal potential impact.
  • No prior incidents related to this deployment.
  • Limited scope reduces potential blast radius.

Recommendation:
Proceed with deployment as planned. Standard monitoring post-deployment is advised to quickly detect any unforeseen issues.


Powered by DriftWatch – Your Deployment Safety Net

@sarahelsagheir

Copy link
Copy Markdown
Author

Hi @munafio, quick update on this PR.

I’ve continued polishing the voice message feature:

  • Voice recordings are sent and rendered as audio messages with the waveform player.

  • Voice attachments now show as “Voice message” with a microphone icon in the conversations list instead of generic “Attachment”.

  • Recording duration is now configurable via CHATIFY_MAX_VOICE_RECORDING_TIME / chatify.php, instead of being hardcoded only in JS.

  • Added broader browser recording support by choosing a supported audio MIME type, including Firefox/Ubuntu-friendly formats.

  • Kept voice-message file types separated in config with allowed_voice_messages.

  • Refactored the contacts query — the old join+groupBy could 500 under strict SQL mode (ONLY_FULL_GROUP_BY); now grouped so it's valid regardless of engine/version. Verified paginated results return 200 with no 1055s.

  • Ran syntax checks on the touched PHP files and voiceMessage.js.

The demo/details are already in the PR above. Happy to adjust the UI text or behavior if you prefer a different naming/style before merge.

@munafio

munafio commented Jul 13, 2026

Copy link
Copy Markdown
Owner

@sarahelsagheir thanks for your contribution and effort, will check it and test asap, and will be informed about any update.

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.

5 participants