YunoHost-Apps / pixelfed_ynh

The federated image shareing service Pixelfed for YunoHost
https://pixelfed.org/
GNU Affero General Public License v3.0
44 stars 15 forks source link

Testing : 0.11.6 (various fixes for v0.11.5 : media/s3/…) #210

Closed lapineige closed 1 year ago

lapineige commented 1 year ago

209 and then v0.11.6 #214

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)

yalh76 commented 1 year ago

!testme

yunohost-bot commented 1 year ago

:v: Test Badge

lapineige commented 1 year ago

It's the very same as https://github.com/YunoHost-Apps/pixelfed_ynh/pull/209#issuecomment-1496224053, nothing change since that time. So I'm cancelling CI. This needs manual testing.

wesleycook74 commented 1 year ago

Edit: Nevermind, I was mistaken. It seemed like it was working but not actually.

I was finally able to fix this issue on my server by updating to this branch and then setting the filesytem config like so

        'local' => [
            'driver' => 'local',
            'root'   => storage_path('app'),
            'permissions' => [
                'file' => [
                    'public' => 0644,
                    'private' => 0660,
                ],
                'dir' => [
                    'public' => 0775,
                    'private' => 0770,
                ],
            ],
        ],
lapineige commented 1 year ago

Edit: Nevermind, I was mistaken. It seemed like it was working but not actually.

Too bad :(

lapineige commented 1 year ago

This adds improvements while not fixing the image upload bug, so the discussion will continue here, but I'm merging it anyway.