YunoHost-Apps / synapse_ynh

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

Config panel no sed #374

Closed Gredin67 closed 11 months ago

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

Gredin67 commented 1 year ago

!testme

yunohost-bot commented 1 year ago

:stuck_out_tongue_winking_eye: Test Badge

Gredin67 commented 1 year ago

@Josue-T what do you think about merging this ? I could test and it works. I just left the settings requiring to handle yaml lists for a later PR.

Gredin67 commented 1 year ago

It seems I messed up something: upgrade defaults to :scream: NOT Backing up the app before upgrading...

Gredin67 commented 1 year ago

!testme Backup before upgrade issue should be solved by last commit

yunohost-bot commented 1 year ago

:rocket: Test Badge

Gredin67 commented 1 year ago

!testme

yunohost-bot commented 1 year ago

Alrighty! Test Badge

Gredin67 commented 1 year ago

@Josue-T do you see something against merging this ?