Ho aggiornato a Discourse 3.5.0.beta6-dev (da0b303568) stamattina.
Dopo l’aggiornamento, mi è stato presentato un banner rosso in cima al sito che diceva:
Avviso amministratore: stai usando il componente del tema discourse-add-groups-to-about. Questa funzionalità è ora disponibile nel core di Discourse. Dovresti rimuovere questo componente del tema.
Successivamente sono andato nelle impostazioni del mio sito Discourse e ho riapplicato tutte le impostazioni della pagina “about” che utilizzavo nel vecchio componente:
No, ma non ho installato il componente. Forse è per questo che la migrazione non è avvenuta per me. Ma non ero molto interessato perché non uso la funzionalità.
Infatti, seguo le PR su GitHub principalmente perché gli screenshot aiutano con la traduzione, specialmente per funzionalità come questa che non sono disponibili nell’interfaccia all’inizio.
È nel core, ma l’impostazione show_additional_about_groups è ancora un’impostazione nascosta del sito, quindi non è visibile.
Penso che la migrazione l’abbia abilitata per i siti che utilizzavano il componente prima. Non sono sicuro se sarà abilitata per tutti o semplicemente rimossa. Penso che sia necessaria solo durante il processo di migrazione.
Se ho capito correttamente, l’idea è che l’impostazione show_additional_about_groups venga aggiunta al core. Quindi viene aggiunto un controllo al componente del tema in modo che i gruppi vengano visualizzati solo se show_additional_about_groups è falso. Ciò significa che continua semplicemente a funzionare come prima. Successivamente, tutto è integrato nel core, ma viene visualizzato solo se show_additional_about_groups è true.
Dopo che le impostazioni sono state migrate automaticamente, show_additional_about_groups viene attivato. Ora il codice del core mostra i gruppi e non più il componente.
Questa migrazione è stata aggiunta 3 giorni fa. Poi, 2 giorni fa, è stata aggiunta l’avviso nel componente che ti è stato mostrato.
Non so perché la migrazione ovviamente non ha funzionato per te (né le impostazioni sono state applicate automaticamente, né show_additional_about_groups è stato attivato).
Penso che sia decisamente una svista il fatto che l’avviso globale non verifichi se l’utente è un amministratore.
Hmm, quindi immagino che sia abilitata per tutti per impostazione predefinita, per tutti coloro che non utilizzavano in precedenza il componente del tema?
E il mio semplicemente non visualizza nessuno dei gruppi nella pagina Informazioni perché la mia impostazione è ancora bloccata su false in background?
Non crederesti a quanti email e richieste abbiamo ricevuto dai nostri membri riguardo a questo banner di avviso
Pensavo che la migrazione avrebbe dovuto spostare la configurazione di @Richie dal componente alle impostazioni principali e avrebbe dovuto anche abilitare l’impostazione nascosta del sito show_additional_about_groups. Un motivo per cui non ha funzionato potrebbe essere che ha cambiato il nome, perché è facilmente fattibile nell’interfaccia.
C’è una ragione per cui la migrazione si basa solo sul nome invece di controllare anche un componente in cui la remote_url dalla tabella remote_themes corrisponde? Ciò catturerebbe i componenti rinominati purché siano stati installati dal repository ufficiale.
L’impostazione nascosta del sito, che non è stata abilitata perché la migrazione non è avvenuta, ha impedito al core di mostrare i gruppi e, invece, il componente mostrava ancora i gruppi. Ma l’avviso globale dal componente gli ha detto di eliminare il componente. Quindi il core non mostrava ancora i gruppi perché show_additional_about_groups era ancora disabilitato e non era facilmente possibile abilitarlo.
Quindi, se la migrazione automatica non ha funzionato, come fanno gli amministratori a migrare manualmente? Copiare la configurazione non è il problema. Ma quando passano dalla visualizzazione dei gruppi del componente alla visualizzazione dei gruppi del core senza dover abilitare l’impostazione nascosta?
Forse sarebbe stato meglio abilitare show_additional_about_groups per tutti o mostrarlo nell’interfaccia prima di aggiungere un avviso al componente che dovrebbe essere eliminato. Quindi una migrazione manuale avrebbe funzionato e il core avrebbe mostrato i gruppi in seguito, quindi l’eliminazione del componente non lascerebbe gli amministratori senza alcun gruppo nella pagina informazioni.
In questo momento, quando un amministratore aggiunge gruppi all’impostazione del sito about page extra groups, non succede nulla perché l’impostazione nascosta show_additional_about_groups non è abilitata. Sembra un bug, anche se dal punto di vista di uno sviluppatore, tutto funziona come previsto. Penso che sarebbe più facile per un amministratore capire cosa succede se l’impostazione fosse visibile invece che nascosta.
Nessuna delle mie impostazioni è stata copiata e, quando le ho copiate, i gruppi non erano ancora visibili nella pagina /about perché l’impostazione nascosta ne impediva la visualizzazione e ho dovuto usare la console Rails per risolvere il problema.
@ted / @kris.kotlarek / @hugh qualche idea in merito? La migrazione fatta da Ted necessita di un perfezionamento?
[quote=“Moin, post:17, topic:368218”]
Al momento, quando un amministratore aggiunge gruppi all’impostazione del sito about page extra groups (gruppi extra della pagina
Abbiamo utilizzato questa impostazione del sito nascosta nel processo di porting del componente del tema. A causa di un problema con la migrazione, non è stata abilitata correttamente quando i dati sono stati migrati.
Ora che il porting è terminato, non abbiamo più bisogno di questa impostazione, come ha sottolineato @sam. Possiamo semplicemente guardare l’elenco dei gruppi configurati.
Ottimo! Pensavo che avrebbe dovuto rimanere fino alla prossima versione stabile per assicurarsi che il componente non mostri i gruppi oltre al core per coloro che eseguono la migrazione.
