YunoHost-Apps / garradin_ynh

Logiciel libre de gestion associative pour YunoHost
https://garradin.eu/
GNU General Public License v3.0
7 stars 8 forks source link

Testing #87

Closed rodinux closed 2 years ago

rodinux commented 2 years ago

Problem

Solution

PR Status

Automatic tests

Automatic tests can be triggered on https://ci-apps-dev.yunohost.org/ after creating the PR, by commenting "!testme", "!gogogadgetoci" or "By the power of systemd, I invoke The Great App CI to test this Pull Request!". (N.B. : for this to work you need to be a member of the Yunohost-Apps organization)

rodinux commented 2 years ago

!testme

yunohost-bot commented 2 years ago

Fingers crossed! Test Badge

rodinux commented 2 years ago

Release version:

Version 1.1.30 (5 septembre 2022)

Restaure les descriptions et comptes usuels qui ont été écrasés par erreur lors de la mise à jour 1.1.29 dans le plan comptable associatif 2018
Corrige le montant restant à payer d'une activité si rien n'a été réglé

Version 1.1.29 (2 septembre 2022)

On peut désormais partager un document (bouton Partager dans la liste des documents)
Dans les pages web, les fichiers SVG sont désormais traités comme des images et non comme des documents
Amélioration du rendu des liens vers les documents dans les pages web
Correctif : lors de la duplication d'écriture de type recette/dépense, les comptes n'étaient pas dupliqués
Les fichiers qui peuvent contenir du code malveillant sont interdits dans les documents (.exe/.com/.php, etc.)
Désactiver la suppression de lien entre écriture et activité depuis la modification d'écriture, permettre de dé-lier entre activité et écriture depuis la liste des écritures d'une activité souscrite
Lors de l'import d'un fichier de compta, si le type d'écriture n'est pas précisé, l'import essaiera de déterminer si c'est une recette ou une dépense (merci @André), sinon le type sera "avancé".
Corrigé : lors du règlement d'une dette/créance, la date était positionnée au début de l'exercice.
Corrigé : parfois les pages web apparaissaient en double dans la recherche après que leur URI ait été modifiée.
rodinux commented 2 years ago

!testme

yunohost-bot commented 2 years ago

:sunflower: Test Badge

rodinux commented 2 years ago

!testme

yunohost-bot commented 2 years ago

:rocket: Test Badge

rodinux commented 2 years ago

!testme

yunohost-bot commented 2 years ago

Meow :cat2: Test Badge

rodinux commented 2 years ago

Je ne comprends pas pourquoi l'installation en Private mode fonctionne dans le premier test et pas ensuite ?? En testant localement avec virtualbox ça fonctionne...

Sinon j'ai enlever un des anciens test dans le check_process qui me semblait désuet...

rodinux commented 2 years ago

Ah ben voilà, les tests sont passés ! Qui peut accepté un review ? @fflorent ? @ericgaspar ? @frju365 ?

Par contre je me pose la question sur - The --keep option of ynh_setup_source expects relative paths, not absolute path ... you do not need to prefix everything with '$final_path' in the --keep arg .... Après avoir tester d'enlever --dest_dir="$final_path sur les lignes 100 et 152 du script upgrade, les tests ne passaient pas.

Sinon que faire avec ce message: Failed to parse apps.json history for at commit 85f303afc3a874ec4487396e8777d20b508c22ff / 2021-11-15 00:00:00 ... ignoring ?

rodinux commented 2 years ago

Merci !