Improve Settings: Remove username and password requirement for ldap#16592
Improve Settings: Remove username and password requirement for ldap#16592snipe merged 1 commit intogrokability:developfrom
Conversation
|
💖 Thanks for this pull request! 💖 We use semantic commit messages to streamline the release process and easily generate changelogs between versions. Before your pull request can be merged, you should update your pull request title to start with a semantic prefix if it doesn't have one already. Examples of commit messages with semantic prefixes:
Things that will help get your PR across the finish line:
We get a lot of pull requests on this repo, so please be patient and we will get back to you as soon as we can. |
PR Summary
|
740fa00 to
5658979
Compare
|
Hi there - thanks for this! Can you please re-target this PR to point to the You usually don't need to close and re-open. After you create a pull request, you can modify the base branch so that the changes in the pull request are compared against a different branch. By changing the base branch of your original pull request rather than opening a new one with the correct base branch, you’ll be able to keep valuable work and discussion. Thanks! |
Since 9d62793 anonymous LDAP login is available. Remove username and password requirement in settings dialog.
|
I've retarget the pull request and rebased the commit onto the develop branch |
|
Congrats on merging your first pull request! 🎉🎉🎉 |

Since 9d62793 anonymous LDAP login is available. Remove username and password requirement in settings dialog.