Skip to content
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

Open
wants to merge 2 commits into
base: doryphore
Choose a base branch
from

Conversation

J9rem
Copy link
Contributor

@J9rem J9rem commented May 24, 2024

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

  • ajoute un test de version pour utiliser UpdateHandler__.php uniquement pour les versions jusqu'à 4.4.4
  • ajoute UpdateAction__.php uniquement pour les versions après 4.4.4 pour ajouter le lien de gestion de l'extension en dessous du bouton de mise à jour des pages d'administration

tag:

  • une fois cette PR intégrée, il serait bon de créer un nouveau tag pour déployer le correctif

pour tester:

  • en doryphore 4.4.4 : handler /update
  • après doryphore 4.4.4 : lien ?GererMisesAJour&action=post_install

Copy link

@mrflos mrflos left a 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`.
Copy link

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)

}
}

}
Copy link

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

Copy link

@mrflos mrflos Aug 1, 2024

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

@J9rem
Copy link
Contributor Author

J9rem commented Aug 1, 2024

@mrflos je trouve tes remarques très pertinentes car :

  • elles réduisent la dette technique
  • elles homogénéisent le code de l'extension avec celui du cœur

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants