-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(UpdateHandler): prepare to next doryphore release #1
base: doryphore
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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?
@@ -7,7 +7,7 @@ Dans la page `GererMisesAJour` de votre YesWiki, recherchez l'extension bénévo | |||
|
|||
## 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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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. |
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__.php
uniquement pour les versions jusqu'à4.4.4
UpdateAction__.php
uniquement pour les versions après4.4.4
pour 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/update
doryphore 4.4.4
: lien?GererMisesAJour&action=post_install