-
Notifications
You must be signed in to change notification settings - Fork 437
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
feat(reactions): Add 👋 and 🔥 call reactions #13345
Conversation
Signed-off-by: Joas Schilling <coding@schilljs.com>
93a88f8
to
22693b9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested
Question arise: how it's supposed to work in federated calls with older versions? UPD: tested with stable30, old version has a fallback emoji, but not a gif. Apart from it, reactions are working properly |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Receiving works fine on iOS. For sending we currently just check the local capabilities and not the remote ones. So that will only work if we change the iOS app or if local server also supports the new emojis.
/backport to stable30 |
☑️ Resolves
🏁 Checklist
docs/
has been updated or is not required