Add missing LDAP dependency - Githubissues

linuxserver / docker-grocy

A container for grocy - the ERP application for your kitchen https://grocy.info

GNU General Public License v3.0

333 stars 44 forks source link

Add missing LDAP dependency #35

Closed mountainsandcode closed 3 years ago

mountainsandcode commented 3 years ago

[X] I have read the contributing guideline and understand that I have made the correct modifications

Description:

Add the php7-ldap library, which is required since the latest version provides LDAP capabilities.

Benefits of this PR and context:

Fixes #32

How Has This Been Tested?

Source / References:

LinuxServer-CI commented 3 years ago

I am a bot, here are the test results for this PR: https://ci-tests.linuxserver.io/lspipepr/grocy/v3.0.1-pkg-b221b660-pr-35/index.html https://ci-tests.linuxserver.io/lspipepr/grocy/v3.0.1-pkg-b221b660-pr-35/shellcheck-result.xml

aptalca commented 3 years ago

Thanks for the PR, but please also add a changelog entry into readme-vars.yml

mountainsandcode commented 3 years ago

Thanks for the PR, but please also add a changelog entry into readme-vars.yml

Done :)

LinuxServer-CI commented 3 years ago

I am a bot, here are the test results for this PR: https://ci-tests.linuxserver.io/lspipepr/grocy/v3.0.1-pkg-b221b660-pr-35/index.html https://ci-tests.linuxserver.io/lspipepr/grocy/v3.0.1-pkg-b221b660-pr-35/shellcheck-result.xml