Événements de suivi effrayants 👻

Il y a des problèmes que je remarque depuis des mois, mais que je n’ai jamais signalés, car je n’arrivais jamais à les reproduire de manière constante. Mais enfin, par accident, je les ai filmés ! (Veuillez donc excuser les pauses et les mouvements de curseur qui témoignent de ma confusion.)

Les points à surveiller sont :

  1. L’onglet « Non lus » vide
  2. Le tag Contribute > Feature affiché comme « Normal » (alors qu’il est suivi, comme indiqué dans mes préférences) et qui ne passe à « Suivi » qu’après un rafraîchissement de la page
  3. Un sujet qui apparaît mystérieusement dans la liste des non lus (datant de plusieurs jours, provenant de la catégorie howto)

Le deuxième point est assez similaire à un bug courant que j’observe (mais que je n’ai jamais pu reproduire en vidéo) : je clique sur un sujet depuis « Nouveaux » dans une catégorie que je suis, mais son statut n’apparaît pas comme « Suivi » tant que je ne rafraîchis pas la page. (Je me demande si cela est lié à ceci.)

Juste avant d’enregistrer cette vidéo, j’essayais de reproduire ce bug en suivant le tag Contribute > Bug, puis en cliquant sur l’un de ses sujets depuis l’onglet « Non lus ». Je n’y suis pas parvenu, mais après avoir désactivé le suivi du tag Contribute > Bug, mon onglet « Non lus » s’est rempli d’environ 20 sujets fermés de Contribute > Bug datant d’environ 2015, ce qui semble être le même bug que le point 3.

Je ne comprends pas pourquoi cela peut se produire. :ghost:

2 « J'aime »

What are the repro steps?

1 « J'aime »

There are two bugs here.

I can’t consistently reproduce the first, where the correct notification level is only shown after a refresh (of a category, or a topic). It’s been happening to me for months, but whenever I try to reproduce it, it doesn’t happen.

The second, which I think is a bit more minor, but still a nuisance can be reproduced (for a limited time only) like so:

  1. Clear /unread
  2. Track a category currently set to ‘Normal’
  3. Go back to /unread and observe posts not previously in it

I’ve been able to do this with a number of categories, but only once for each. Once I’ve dismissed the posts from unread, they never appear again.

Consistently reproducing it takes a little more work, so hold onto your hats, it’s gonna get a bit crazy:

  • Go to unread in a category of your choosing which has a notification level of “Normal”, observe nothing there:

  • List all the currently tracked topics in that category (and remember, or take a screenshot of them):

  • Enter a topic from that category not on that list:

  • Start tracking the category
  • Again list tracked topics in that category
  • Observe the topic you just opened in that list:

  • Go to unread, and again observe that topic in the list:


TL;DR

So, the bug here is that previously opened topics in a category become tracked, and appear in unread (unless previously dismissed), when the category is tracked, whereas nothing happens to old not-opened-before topics.

To me, what should happen is either:

  • Nothing changes about the notification status of old topics in a category when it becomes tracked, previously opened or not.

or

  • All old topics become tracked when the category is tracked, but are automatically dismissed, so only new activity appears in the unread tab.

I lean towards the second solution, provided explicitly set notification levels on topics are kept.

I haven’t investigated how this works with other notification levels, I imagine its the same, but I feel it would confuse things for the time being.

So in relation to

What part is considered as “not working as expected”?

1 « J'aime »

2 parties :

La première concerne les sujets qui ne sont pas correctement suivis implicitement. S’ils l’étaient, tous les sujets de Community Building > Praise devraient apparaître à cette étape :

Il semble que cette liste affiche uniquement les sujets ayant un historique de suivi marqué comme « tracked » (suivi), ce qui ne fonctionne pas comme prévu. Le comportement attendu serait d’afficher tous les sujets ayant un historique de suivi marqué comme « tracked », ainsi que tous les sujets appartenant à une catégorie suivie. (La même logique s’appliquerait pour les abonnements.)

Par ailleurs, bien que cela soit logique :

Il semble que la même fonctionnalité fasse défaut lorsqu’un utilisateur suit une catégorie. Ne devrait-on pas également créer un enregistrement de suivi lorsqu’un nouveau message est publié dans un sujet ?

La seconde concerne l’apparition de sujets apparemment arbitraires comme non lus lorsque l’on suit une catégorie (je suppose qu’il s’agit d’un effet secondaire du changement d’état de suivi des anciens sujets, passant de « regular » à « tracked »), ce qui pourrait être corrigé en les marquant automatiquement comme lus.

What are your Preferences → Other
Consider topics new when
and
Automatically track topics I enter
settings?

I haven’t viewed them yet

never

1 « J'aime »

C’est un problème que j’ai découvert il y a quelque temps, et un fil de discussion à ce sujet est disponible ici (je ne suis pas tout à fait sûr de la raison pour laquelle il a été déplacé de Contribute > Bug vers Contribute > Feature).

Le fil est assez confus car, à l’origine, je pensais que le problème était causé par l’import initial depuis phpBB. Voici donc une tentative de résumé :

Tout se résume au fait qu’un sujet ne peut pas être à la fois nouveau et non lu simultanément :

  • Bien qu’il soit implicitement suivi, le sujet n’apparaît pas dans non lu car l’utilisateur ne l’a jamais consulté.
  • Le sujet n’apparaît pas dans nouveau car il dépasse le seuil défini par votre paramètre « Considérer les sujets comme nouveaux pendant… ».

C’est un problème difficile à résoudre, car faire apparaître chaque ancien sujet d’une catégorie dans votre onglet « non lu » n’est vraiment pas une bonne expérience utilisateur.

Les « dissiper automatiquement » est assez coûteux en termes de base de données (un nouvel enregistrement TopicUser devrait être créé pour chaque sujet de la catégorie que vous venez de suivre).

Nous n’avons pas vraiment trouvé d’idée élégante pour résoudre ce problème, et la discussion s’est quelque peu essoufflée. Une option proposée par @sam était la suivante :

À l’époque, je pensais que cette idée pourrait être bonne :

Mais

TL;DR : Je n’ai pas de solution, mais j’espère que cela clarifie le problème :wink:

3 « J'aime »

And for the other issue in this thread:

I can reproduce this consistently here on meta for categories. Repro steps:

  1. Go to your user preferences, remove all tracked categories
  2. Go to a category page, e.g. #meta, and observe that the ‘notification level’ indicator is “normal”
  3. Go immediately to your user preferences, by clicking your avatar in the top right, then the cog
  4. Add the category you just visited, e.g. #meta, to your “tracking” categories
  5. Press “save changes”
  6. Press the back button in your browser to return to #meta, and observe that the ‘notification level’ indicator is still “normal”
  7. Refresh the page, and observe that the ‘notification level’ is now “tracked”

If you replace step 6 with visiting the category page by some other route, the tracking status appears correctly.

5 « J'aime »

From a practical perspective, that doesn’t make sense to me. If I buy a book and put it on my shelf having not opened it, it is both new and unread.

Wouldn’t it be possible to have an old topic (with no tracking record) only appear in unread when there’s new activity on it?

So, just as a tracking record is created on new activity for topics in watched categories, a tracking record would be created on new activity for topics in tracked categories.

Combined with the above, couldn’t we just dismiss topics where the tracking record has been changed to “tracking” as a result of tracking the category, which would mean the same user experience, without any additional database writes? (If I understand how tracking records, dismissing and TopicUser relate.)

This way, it seems to me, we could have a mailing list experience for tracking too! :smile:

Sure, I agree. I was talking purely about how it currently works in discourse.

I believe @sam thought this would have major performance issues

I’m 99% sure a new record isn’t created for tracked topics - @sam said “created when we notify users”, and tracking doesn’t cause notifications. Maybe fixing this is the solution :slight_smile:

1 « J'aime »

Do you know if there’s an architectural issue which means the new/unread either/or has to be the case?

Indeed, I was proposing this as a solution which seems to not have any major performance issues.

1 « J'aime »

I assume it’s by design - having a topic listed on the new tab and the unread tab would be a bit weird.

Taking your book analogy, while it can be both unread and new, you can’t put it on your “new” shelf and your “unread” shelf simultaneously :wink:

4 « J'aime »

Ok… I’ve done some testing in my dev environment. As a simplest possible repro:

  1. Set previous visit timeout hours site setting to 0, to make testing quicker
  • User A sets “Consider topics new when” to “created since I was last here”
  • User A sets Category1 to tracked
  • User B posts in Category1
  • User A observes the topic in ‘new’, but does not open it
  • User A leaves the site
  • User A comes back to the site later, and User B’s post is gone from the ‘new’ tab
  • User B posts again in the same topic
  • The topic does not appear in either ‘new’ or ‘unread’ for User A, even though they are tracking the topic

That same repro works for ‘watching’ a category as well. A notification is sent, but no TopicUser row is created (checked on the rails console). It seems that this:

is not happening for some reason, both for watched and tracked categories.

1 « J'aime »

Would it though? We already have topics listed in the latest tab and the new/unread tabs simultaneously. And the two (new and unread) are for different purposes: new is “show me the newest topics on this site” and unread is “show me topics I’ve expressed an interest in, but haven’t (finished) read(ing)”. For new to achieve its purpose it has to have unread topics in it which are new, and for unread to achieve its purpose it needs to have new topics in it which are unread.

Currently the unread tab is rather dysfunctional.

Interesting, thanks for doing all this investigation! So it would seem like if this did actually happen (or more specifically, a TopicUser row was created on topic update, rather than notification, to include tracked categories), then our problems here would be solved. @sam thoughts?

I am fine to change the semantics of “tracked category” so it means, if a category is tracked it will unconditionally show up in new regardless of your new duration, it does not mean this now.

4 « J'aime »