Don
Novembre 13, 2024, 6:23
1
Bonjour
J’ai trouvé quelques endroits où je pense que la présence en ligne devrait également être masquée.
Présence dans le chat, lorsque vous écrivez un message, elle s’affiche en bas même si vous masquez votre présence.
Je pense que sur le résumé de la page utilisateur, dans les statistiques, la section “Vu” devrait également être masquée pour les utilisateurs si l’option “Masquer la présence” est activée.
Il serait également bon que cela fonctionne comme le point 2 avec le composant de thème “Dernière présence” Carte utilisateur .
Merci
4 « J'aime »
lindsey
Novembre 21, 2024, 3:35
2
Merci d’avoir partagé ces idées ! Je vais déplacer ceci dans Feature
1 « J'aime »
Don:
Présence dans le chat, lorsque vous écrivez un message, elle s’affiche en bas même si vous masquez votre présence.
Cela sera corrigé par
main ← fix-is-replying-when-offline
opened 06:57PM - 03 Dec 24 UTC
While trying to reproduce an issue reported internally where a user saw the "is … replying..." from another user who had disabled presence (aka. was "offline"), I noticed something odd about the way we "notify" presence via `notifyState`. The way it is setup up, it's being called on every "input" event in the composer/editor, due to it tracking the "reply" property of the composer model. That's fine, but the way the `helperFn` helper works, make it so we're calling the `on.cleanup` callback also on **every** "input" events in the composer/editor. For every character we enter in the composer, we're "entering" the presence channel and immediatelly "leaving" it...
If I remove the checks on `reply` (and thus the auto-tracking), then the `notifyState` function is called much less frequently. But we have some specs that expects us not to "publish" our presence when we open the composer and the reply is "empty".
Note: I switched to using `replyDirty` instead to re-use the checks the composer model is already doing to ensure we actually have changed something in the reply.
The best I came up with was not to rely on the `helperFn` but now, there's no way (that I could find) to call `composerPresenceManager.leave()` when the composer is closed. I tried a lot of different ways, but none worked. I'm sure it's something stupid easy, but I can't figure it out just now.
Internal ref - t/127490/74
(une fois que j’aurai réussi à le faire fonctionner correctement
2 « J'aime »
)