YunoHost-Apps / synapse_ynh

Matrix server (synapse) package for YunoHost
https://matrix.org/
GNU General Public License v3.0
79 stars 42 forks source link

Fix issue with user creation script #359

Closed csolisr closed 1 year ago

csolisr commented 1 year 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)

Josue-T commented 1 year ago

!testme

yunohost-bot commented 1 year ago

Meow :cat2: Test Badge

Josue-T commented 1 year ago

Hello, thanks for the patch but we still need to keep the retro-compatibility with the current yunohost version, so you might need to add something like if ynh_version > x, else....

csolisr commented 1 year ago

Hello, thanks for the patch but we still need to keep the retro-compatibility with the current yunohost version, so you might need to add something like if ynh_version > x, else....

Yeah I thought something like this might happen... BRB checking how to fetch the current version of YNH.

csolisr commented 1 year ago

As per this commit the YNH version must be >= 11.1

Josue-T commented 1 year ago

!testme

yunohost-bot commented 1 year ago

May the CI gods be with you! Test Badge