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

feat(conversation): add federated typing indicator #14443

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

miaulalala
Copy link
Contributor

@miaulalala miaulalala commented Feb 17, 2025

☑️ Resolves

🛠️ API Checklist

🚧 Tasks

  • ...

🏁 Checklist

  • ⛑️ Tests (unit and/or integration) are included or not possible
  • 📘 API documentation in docs/ has been updated or is not required
  • 🔖 Capability is added or not needed

@miaulalala miaulalala added this to the 🪺 Next Major (32) milestone Feb 17, 2025
@miaulalala miaulalala self-assigned this Feb 17, 2025
@nickvergessen nickvergessen force-pushed the feat/federated-typing-indicators branch from da6264e to 722e190 Compare February 19, 2025 11:21
Signed-off-by: Anna Larch <anna@nextcloud.com>
@miaulalala miaulalala force-pushed the feat/federated-typing-indicators branch from 722e190 to edec72e Compare February 19, 2025 13:12
@miaulalala miaulalala marked this pull request as ready for review February 19, 2025 13:52
@@ -40,7 +40,6 @@ public function overwrittenRemoteTalkHash(string $hash): string {
'config' => [
'chat' => [
'read-privacy',
'typing-privacy',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's still overwritten, just not with a hardcoded value anymore, but the value might not be what the host send, so we should keep that.
In fact we might want to add the value in here, so when you change your setting the ones of the room also change?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

Typing indicator privacy is always set to private for federated rooms
2 participants