Tracciare un argomento con la scorciatoia da tastiera ne traccia anche altri

Contribuisci Bug
1

Quando utilizzo le scorciatoie m t e m r per seguire/non seguire un argomento, questo modifica anche lo stato di tracciamento di altri argomenti che ho visualizzato di recente nella stessa finestra.

Questo non sembra accadere quando si usa il mouse.

Osservato utilizzando Firefox 68.4.1esr. Notato per la prima volta su Rust Discourse, ora l’ho verificato anche qui.

5 Mi Piace
3

Ciao, grazie per la segnalazione!

Penso che questo bug sia presente da un bel po’ di tempo; forse i recenti cambiamenti lo hanno reso più evidente.

Per farla breve:

  • non stiamo validando l’ID del topic quando riceviamo l’evento di modifica
  • non stiamo applicando un throttling alla scorciatoia da tastiera, il che significa che se tieni premuto il tasto “m”, l’evento viene inviato all’infinito…

Questo dovrebbe risolvere il problema:

Dovrebbe essere unito entro la fine della settimana.

7 Mi Piace
4

Ciao,
quanto tempo dovrebbe passare dalla fusione della pull request al deployment della correzione?
Al momento non osservo alcun cambiamento nel comportamento qui.

5

Dipende da quale forum. Il commit è stato unito nel master e retroportato nel ramo stabile, quindi tutti dovrebbero poterlo ottenere.

4 Mi Piace
6

Questo argomento è stato automaticamente chiuso dopo 3 giorni. Non sono più consentite nuove risposte.

8

@Falco è tornato indietro?

2 Mi Piace
9

Ciao @david
Posso ancora riprodurre il problema su questo forum. Non so se nel frattempo sia diventato irriproducibile.

2 Mi Piace
10

@joffreyjaffeux could you take another look please?

12

@mpol non siamo riusciti a riprodurre il problema. Potresti illustrarci i passaggi esatti che segui per riprodurlo qui su Meta?

1 Mi Piace
13

Mi scuso se si tratta di un problema da parte mia, ma eccoci qui, su Firefox 68.6.0esr proprio ora.

  1. Utilizzando solo la navigazione da tastiera, partendo da una pagina principale appena aperta https://meta.discourse.org/ in una nuova scheda, all’inizio non ho argomenti monitorati.
  2. Premo ‘j’, ‘Invio’ per entrare in un argomento.
  3. Premo ‘j’ più volte fino agli argomenti suggeriti, poi ‘Invio’ per entrarne in uno.
  4. Premo ‘j’, ‘m’, ‘t’ - ora sto monitorando sia questo argomento che quello entrato dalla pagina principale.

A proposito, ho appena notato che quando al passaggio 3 ho premuto ‘m’, ‘t’ immediatamente, solo l’argomento corretto è stato monitorato. Ma anche dopo, premendo ‘j’, ‘m’, ‘t’ ha cambiato lo stato dell’argomento precedente.

1 Mi Piace
14

Quindi non sono riuscito a riprodurre questo bug esatto e nemmeno @david, ma @david aveva molte richieste in corso nel suo browser, cosa che non avevo io. La prima idea è stata il browser e poi più browser aperti?

Dopo una ricerca ho capito che il problema proveniva dalle dimensioni del browser (che per me era stretto e per David era grande) e dal pulsante della cronologia dei topic che continuava a registrare nuovi gestori appEvents, visibile per David e non per me.

Penso che questo dovrebbe migliorare notevolmente la situazione sul tracciamento dei topic, maggiori dettagli nel commit:

4 Mi Piace
15

Ah, sì, le richieste. Quindi alla fine la procedura sopra ha generato due richieste POST identiche a https://meta.discourse.org/t/146924/notifications e poi quella, presumibilmente problematica, a https://meta.discourse.org/t/113192/notifications.

2 Mi Piace
16

Sì, questo dovrebbe risolvere questo tipo di problemi. Lo sto unendo ora, così potrai provarlo su Meta a breve.

5 Mi Piace
17

@joffreyjaffeux Grazie. Dopo un paio di tentativi non sono più riuscito a riprodurre il problema qui e, infatti, ora vedo sempre una sola e corretta richiesta a /notifications: nessun duplicato e nessuna richiesta per altri argomenti.

3 Mi Piace