Ho aperto una PR per consentire agli utenti anonimi di mettere “mi piace” ai post. È un’impostazione del sito (SiteSetting) in modo che gli amministratori abbiano la possibilità di abilitare o disabilitare la funzionalità.
Potrei per favore ricevere una revisione del codice?
Contesto: Faccio parte di un’organizzazione 501(c)3 che utilizza Discourse, dove i nostri account sono registrati con i nostri nomi reali. Incoraggiamo la discussione anonima, poiché alcune persone vogliono condividere esperienze personali e argomenti sensibili (salute mentale, ricordi difficili del passato, ecc.). La discussione anonima è stata estremamente ricca e utile, e l’aggiunta di una funzionalità di “mi piace” anonima aiuterebbe a valutare l’interazione della community su argomenti in cui le persone potrebbero provare una forte empatia o accordo, ma non si sentirebbero a proprio agio a mettere il proprio nome in evidenza.
Potrei avere un rapido consiglio su dove invocare l’icona di disallow? Ho dato un’altra occhiata ma non riesco a capirlo.
Dal commento di GH:
Ciao @nattsw, grazie per la seconda revisione. Ho qualche difficoltà a capire dove dovrebbe avvenire il disallow poiché post_undo_action_window_mins viene fatto riferimento solo in can_delete_post_action?, e il metodo restituisce correttamente false una volta trascorso post_undo_action_window_mins.
Tuttavia, nonostante questo metodo restituisca correttamente false dopo la chiusura della finestra, il simbolo di disallow non appare, ma non c’è nessun altro metodo che faccia riferimento a post_undo_action_window_mins. Mi rendo conto che can_delete_post_action? viene chiamato al caricamento della pagina, ma non riesco a individuare dove venga invocata l’effettiva icona di disallow.
Potresti indicarmi il blocco di codice che chiama il simbolo di disallow? Grazie!
Grazie per il suggerimento! È stato molto utile! Ho applicato una patch, verificato manualmente il comportamento e aggiunto una copertura di test aggiuntiva. È pronto per un altro giro di revisione quando ne avrai la possibilità.
