search

YunoHost-Apps / seafile_ynh

Seafile package for YunoHost
https://seafile.com
MIT License
14 stars 19 forks source link

Testing #78

Closed alexAubin closed 2 years ago

alexAubin commented 3 years ago

Problem

I'm creating this PR because I realize while reading Bullseye's tests that this app has a shitload of commits on testing not merged in master

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)

alexAubin commented 3 years ago

!testme

yunohost-bot commented 3 years ago

Alrighty! Test Badge

HugoPoi commented 3 years ago

This fix multiple issues with yunohost 4.x permissions feature. There is some folder permission issues for the seahub/media.

My workaround.

chmod a+rx /opt/yunohost/seafile
chmod a+rx /opt/yunohost/seafile/seafile-server-7.1.5
chmod a+rx /opt/yunohost/seafile/seafile-server-7.1.5/seahub
chmod -R a+rx /opt/yunohost/seafile/seafile-server-7.1.5/seahub/media
Josue-T commented 3 years ago

My workaround.

chmod a+rx /opt/yunohost/seafile
chmod a+rx /opt/yunohost/seafile/seafile-server-7.1.5
chmod a+rx /opt/yunohost/seafile/seafile-server-7.1.5/seahub
chmod -R a+rx /opt/yunohost/seafile/seafile-server-7.1.5/seahub/media

The workaround on the 8.x branch would be better.

The issue is that I had many issues on the 7.1 version so I prefered to jump directly to the version 8.0 but I still have an issue with LDAP (https://forum.seafile.com/t/ldap-broken-under-seafile-8-0-1-server/13526/23)

Josue-T commented 3 years ago

In 7 days we can merge this

alexAubin commented 2 years ago

@Josue-T : shall we merge ? :stuck_out_tongue_winking_eye:

yalh76 commented 2 years ago

!testme

yunohost-bot commented 2 years ago

:carousel_horse: Test Badge