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

Several refactorations #73

Closed fflorent closed 2 years ago

fflorent 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)

fflorent commented 2 years ago

!testme

yunohost-bot commented 2 years ago

:carousel_horse: Test Badge

rodinux commented 2 years ago

what about the warning ?

! General stuff, misc helper usage

    ! ynh_install_app_dependencies should also be in restore script 
fflorent commented 2 years ago

what about the warning ?

! General stuff, misc helper usage

    ! ynh_install_app_dependencies should also be in restore script 

That's a good point! I'll fix that today, thanks!

rodinux commented 2 years ago

I have try your code on a virtualbox yunohost, I think it nice ! Thanks for your job.

rodinux commented 2 years ago

There is something I can't explain. First I have try install garradin, the first installation in the configuration page said me I have an 1.1.12 version, but it should have been an 1.1.14... I can't understand why ???

fflorent commented 2 years ago

There is something I can't explain. First I have try install garradin, the first installation in the configuration page said me I have an 1.1.12 version, but it should have been an 1.1.14... I can't understand why ???

I get this: 2022-01-02_13-13

fflorent commented 2 years ago

!testme

yunohost-bot commented 2 years ago

Fingers crossed! Test Badge

yunohost-bot commented 2 years ago

:v: Test Badge

rodinux commented 2 years ago

There is something I can't explain. First I have try install garradin, the first installation in the configuration page said me I have an 1.1.12 version, but it should have been an 1.1.14... I can't understand why ???

I get this: 2022-01-02_13-13

Me too after upgrading with your code. I mean with the first install which may install the master branch, I get version 1.1.12 is installed, but it should be 1.1.14 ???

rodinux commented 2 years ago

Ok, we wait for the results, but I think I will push your PR soon, it makes sense.