fix(UpdateHandler): prepare to next doryphore release#1
fix(UpdateHandler): prepare to next doryphore release#1J9rem wants to merge 2 commits intoYesWiki:doryphorefrom
Conversation
mrflos
left a comment
mrflos
left a comment
There was a problem hiding this comment.
Hello, ca serait bien de profiter de cette nouvelle version pour plutot utiliser les migrations pour creer les pages , forms, listes et tables nécessaires et ne plus passer par un handler de gestion exotique, qu'en penses tu?
| ## Utilisation | ||
|
|
||
| Connecté en tant qu'administrateur, administratrice du YesWiki, cliquer sur le bouton `Gérer l'extension bénévolat` en fin de mise à jour (handler `/update`). Vous pouvez retrouver cette interface de gestion avec le handler `/gestionbenevolat`. | ||
| Connecté en tant qu'administrateur, administratrice du YesWiki, cliquer sur le bouton `Gérer l'extension bénévolat` en fin de mise à jour (lien `?GererMisesAJour&action=post_install`). Vous pouvez retrouver cette interface de gestion avec le handler `/gestionbenevolat`. |
There was a problem hiding this comment.
Passer par un handler pour gérer la post-install est un peu "bancal" : si l'usagere ne lit pas la doc, elle ne connait pas l'existence de ce handler, et ne pourra pas finaliser la mise a jour de son yeswiki.
Je pense que ce serait mieux de ne pas permettre de choisir ses noms de pages, mais d'avoir une convention de nommage des pages ou on est presque surs que personne ne reprend ses nom et le les creer directement dans une migration.
(dans les versions futures de yeswiki on pourra sans doute reserver des routes comme pour les api)
| } | ||
| } | ||
|
|
||
| } |
There was a problem hiding this comment.
il vaut mieux pas intervenir sur la page de post install, les migrations ont été créées pour etre lancées en fin d'install et maj, afin d'eviter des clics qui sont parfois oubliés par les usager.es et c'est pas forcement le bon endroit pour rajouter des boutons car tres peu visibles
There was a problem hiding this comment.
Peut être pas les supprimer avant ectoplasme, pour permettre d'installer sur des versions plus anciennes
|
@mrflos je trouve tes remarques très pertinentes car :
Bon par contre, c'est un gros travail de tout réécrire que je ne peux supporter sans financement. Ça me va donc si la PR végète quelque temps en l'état ou si elle est close. On peut laisser le code tel qu'il est et dire que l'extension ne fonctionne plus de façon optimale pour les versions ultérieures de YesWiki. Peut-être que ça motivera les usagers à soutenir financièrement sa maintenance. |
2 participants
Objectif
Permettre la compatibilité de cette extension avec les nouveautés du système de mise à jour de la prochaine version de doryphore
ce que ça fait
UpdateHandler__.phpuniquement pour les versions jusqu'à4.4.4UpdateAction__.phpuniquement pour les versions après4.4.4pour ajouter le lien de gestion de l'extension en dessous du bouton de mise à jour des pages d'administrationtag:
pour tester:
doryphore 4.4.4: handler/updatedoryphore 4.4.4: lien?GererMisesAJour&action=post_install