Testing - Githubissues

YunoHost-Apps / piwigo_ynh

Piwigo package for YunoHost

http://piwigo.org/

GNU General Public License v2.0

6 stars 4 forks source link

Testing #46

Closed JimboJoe closed 4 years ago

JimboJoe commented 4 years ago

Problem

The LDAP plugin installed with this package is obsolete. If upgraded via Piwigo, it leads to a broken authentication as the newer version uses MySQL for configuration instead of a plain file.

Solution

Use latest plugin version
Implement Piwigo Web Services API calls to activate it, in particular to have the internal configuration MySQL table created

(PR #884 on ynh_local_curl opened on core)

PR Status

[ ] Code finished.
[ ] Tested with Package_check.
[ ] Fix or enhancement tested.
[ ] Upgrade from last version tested.
[ ] Can be reviewed and tested.

Validation

Minor decision

Upgrade previous version :
[x] Code review : Maniack C
[x] Approval (LGTM) : Maniack C
[x] Approval (LGTM) : Kay0u
CI succeeded :
When the PR is marked as ready to merge, you have to wait for 3 days before really merging it.

kay0u commented 4 years ago

Can be merged in 3 days