YunoHost-Apps / kresus_ynh

Kresus (personal finance manager) for YunoHost
https://kresus.org/
GNU Affero General Public License v3.0
20 stars 20 forks source link

send_readme_to_admin : wrong choice #68

Closed fxleroy closed 4 years ago

fxleroy commented 4 years ago

Bonjour, Après l'upgrade de kresus 0.16.0.ynh2.xxx en 0.16.0.ynh2.xxx (je ne me souviens pas de la valeur apres le ynh2 qui a changée), les logs affiche le message suivant :

image

Il s'agit du fichier _common.sh ligne 52.

Est ce qu'il faudrait corriger $(yunohost diagnosis show | grep -B 100 "services:" | sed '/services:/d')" en $(yunohost tools diagnosis show | grep -B 100 "services:" | sed '/services:/d')"

De plus, le retour de cette commande ne semble pas utilisé.

nicofrand commented 4 years ago

Salut,

le problème est qu'à partir de YunoHost 3.8 ce sera bien yunohost diagnosis show. Il faut donc soit attendre la 3.8 soit avoir une commande qui fonctionne selon la version.

Le retour de cette commande est directement inséré dans l'email envoyé.

nicofrand commented 4 years ago

Le résultat de cette commande a également changé, je vais plutôt étudier si c'est réellement utile dans l'email. Voire si l'email lui-même est réellement utile.

fxleroy commented 4 years ago

Les logs de l'upgrade sont bien maintenant après les différents commit fait.

Merci