J’ai mis à niveau vers Discourse 3.5.0.beta6-dev (da0b303568) ce matin.
Après la mise à jour, j’ai été accueilli par une bannière rouge en haut du site indiquant :
Avis administrateur : vous utilisez le composant de thème discourse-add-groups-to-about. Cette fonctionnalité est désormais disponible dans le cœur de Discourse. Vous devriez supprimer ce composant de thème.
Ensuite, je suis allé dans les paramètres de mon site Discourse et j’ai réappliqué tous les paramètres de la page “À propos” que j’utilisais dans l’ancien composant :
Je ne peux pas réinstaller le thème maintenant car la grande bannière d’avertissement rouge est visible par tous les utilisateurs, même ceux qui ne sont pas connectés
Non, mais je n’ai pas installé le composant. C’est peut-être pour ça que la migration ne s’est pas faite pour moi. Mais je n’étais pas vraiment intéressé car je n’utilise pas la fonctionnalité.
En fait, je suis les PRs sur GitHub principalement parce que les captures d’écran aident à la traduction, surtout pour les fonctionnalités comme celle-ci qui ne sont pas disponibles dans l’interface au début.
C’est dans le cœur, mais le paramètre show_additional_about_groups est toujours un paramètre de site caché, donc il n’est pas visible.
Je pense que la migration l’a activé pour les sites qui utilisaient le composant auparavant. Je ne sais pas s’il sera activé pour tout le monde ou simplement supprimé. Je pense qu’il n’est nécessaire que pendant le processus de migration.
Si j’ai bien compris, l’idée est que le paramètre show_additional_about_groups est ajouté au cœur. Ensuite, une vérification est ajoutée au composant de thème afin que les groupes ne soient affichés que si show_additional_about_groups est faux. Cela signifie qu’il continue simplement de fonctionner comme avant. Ensuite, tout est intégré au cœur, mais n’est affiché que si show_additional_about_groups est vrai.
Une fois les paramètres automatiquement migrés, show_additional_about_groups est activé. Maintenant, le code du cœur affiche les groupes et non plus le composant.
Cette migration a été ajoutée il y a 3 jours. Puis, il y a 2 jours, la notification qui vous a été affichée a été ajoutée dans le composant.
Je ne sais pas pourquoi la migration n’a manifestement pas fonctionné pour vous (ni les paramètres n’ont été appliqués automatiquement, ni show_additional_about_groups n’a été activé).
Je pense que c’est définitivement un oubli que l’avertissement global ne vérifie pas si l’utilisateur est un administrateur.
Je pensais que la migration aurait dû déplacer la configuration de @Richie du composant vers les paramètres principaux et aurait également dû activer le paramètre de site caché show_additional_about_groups. Une raison pour laquelle cela n’a pas fonctionné pourrait être qu’il a changé le nom, car c’est facilement faisable dans l’interface.
Y a-t-il une raison pour laquelle la migration repose uniquement sur le nom au lieu de également vérifier s’il existe un composant où l’remote_url de la table remote_themes correspond ? Cela permettrait de détecter les composants renommés tant qu’ils ont été installés depuis le dépôt officiel.
Le paramètre de site caché, qui n’a pas été activé car la migration n’a pas eu lieu, a empêché le noyau d’afficher les groupes, et à la place, le composant affichait toujours les groupes. Mais la notification globale du composant lui disait de supprimer le composant. Ensuite, le noyau n’affichait toujours pas les groupes car show_additional_about_groups était toujours désactivé, et il n’était pas facile de l’activer.
Donc, si la migration automatique n’a pas fonctionné, comment les administrateurs migrent-ils manuellement ? Copier la configuration n’est pas le problème. Mais quand passent-ils de l’affichage des groupes par le composant à l’affichage des groupes par le noyau sans avoir besoin d’activer le paramètre caché ?
Peut-être qu’il aurait été préférable d’activer show_additional_about_groups pour tout le monde ou de l’afficher dans l’interface avant d’ajouter une notification au composant indiquant qu’il devrait être supprimé. Ensuite, une migration manuelle aurait fonctionné, et le noyau afficherait les groupes par la suite, de sorte que la suppression du composant ne laisserait pas les administrateurs sans aucun groupe sur la page « À propos ».
En ce moment, lorsqu’un administrateur ajoute des groupes au paramètre de site about page extra groups, rien ne se passe car le paramètre caché show_additional_about_groups n’est pas activé. Cela ressemble à un bug, même si du point de vue d’un développeur, tout fonctionne comme prévu. Je pense qu’il serait plus facile pour un administrateur de comprendre ce qui se passe si le paramètre était visible au lieu d’être caché.
Aucun de mes paramètres n’a été copié et, lorsque je les ai copiés, les groupes n’étaient toujours pas visibles sur la page /about car le paramètre caché les empêchait d’apparaître et j’ai dû utiliser la console Rails pour résoudre le problème.
Nous avons utilisé ce paramètre de site caché lors du portage du composant de thème. En raison d’un problème lors de la migration, il n’a pas été activé correctement lorsque les données ont été migrées.
Maintenant que le portage est terminé, nous n’avons plus vraiment besoin de ce paramètre, comme l’a souligné @sam. Nous pouvons simplement regarder la liste des groupes configurés.
Super ! Je pensais qu’il faudrait attendre la prochaine version stable pour s’assurer que le composant n’affiche pas les groupes en plus du noyau pour ceux qui migrent alors.
