Permessi granulari basati su gruppi per utenti anonimi e registrati

[martin (Martin Brennan)]

@Moin

@martin

Ho approvato comunque la tua PR per il componente, penso che vada bene

C’è un motivo per cui non è ancora stata unita?

No, per qualche motivo immagino che il mio cervello abbia pensato che non si trattasse di una PR nell’organizzazione di Discourse . La unirò subito dopo l’esecuzione dei controlli CI.

@Moin

L’ho fatto. Ma ho ancora l’impressione che aiuti solo per i forum che aggiungono il componente dopo che ho apportato questa modifica. Per quelli che lo utilizzano già, il nuovo valore predefinito non viene applicato (il che di solito è positivo!). Quindi vedo ancora il problema di un cambiamento imprevisto nel comportamento per chi lo utilizza già.

Inoltre, sai cosa succede se un amministratore ha configurato un’impostazione con un gruppo che aggiungo come gruppo non consentito nel mio aggiornamento?

Per questi e altri casi futuri, penso che la soluzione migliore sia probabilmente scrivere una migrazione su base tema/componente Migrate Discourse theme settings