Testing

[ericgaspar]

Problem

• Description of why you made this PR

Solution

• And how do you fix that problem

PR Status

• Code finished and ready to be reviewed/tested
• The fix/enhancement were manually tested (if applicable)

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)

[ericgaspar]

!testme

[yunohost-bot]

🎠

[Josue-T]

Well why did you disable multi instance mode ?

[ericgaspar]

I suspect multi-install teste set to 0 because the multi-install doesn't work (CI failes)?

synapse_ynh/check_process

Line 18 in a1c7e5a

multi_instance=0

[ericgaspar]

!testme

[yunohost-bot]

🚀

[Gredin67]

can we merge this in testing #356 and further test before merging in master ?

[Gredin67]

and probably too long, but would be nice to move to packaging v2 sometime soon #401

[Josue-T]

I suspect multi-install teste set to 0 because the multi-install doesn't work (CI failes)?

synapse_ynh/check_process

Line 18 in a1c7e5a

multi_instance=0

Well I see. I think we should temporary disable the multi instance test but keep enabled in the manifest.json (if possible). And when we migrate to packaging v2 find a way to enable again multi-instance tests.

Anyway we can't remove muti-instance flag in manifest.json it wight break multi-instance installation.

[ericgaspar]

!testme

[yunohost-bot]

😜