zulip/zulip-mobile

Don't send typing notifications unless `send_private_typing_notifications` setting is enabled.

Open

#5.042 aberto em 8 de out. de 2021

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)JavaScript (673 forks)github user discovery
P1 high-priorityhelp wantedwebapp parity

Métricas do repositório

Stars
 (1.348 stars)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

In https://github.com/zulip/zulip/pull/19200, we've added new privacy settings for whether to send typing notifications.

The server will reject typing notification requests if this setting is disabled, so it won't be a user-facing bug to ignore this unless the client-side error handling causes that to be the case. I'll be deploying the feature to CZO soon.

(Note that we're also adding send_stream_typing_notifications, but stream-level typing notifications are not fully implemented yet -- though I think it's mostly client code that's missing and the server implementation is complete). See #4702 for more on that.

Guia do colaborador

Direção de pesquisa
Examine o código do cliente que envia notificações de digitação e adicione uma verificação condicional para a configuração do usuário `send private typing notifications`. Além disso, trate as respostas de erro do servidor quando a configuração estiver desabilitada.
Pilha de tecnologia
javascriptreact native
Domain
mobile
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
GitJavaScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70