Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stable15, behaviour with empty LDAP user-attributes undocumented #1008

Closed
wants to merge 37 commits into from

Conversation

victorbw
Copy link

@victorbw victorbw commented Dec 5, 2018

As mentioned here #12582, there's no documentation about what happens when LDAP user-attributes are left empty:

grafik

Sugestion for somewhere in the top chapter of LDAP-documentation:

Note

When a LDAP object's name or surname is left empty, Nextcloud will treat it as an empty object and will not return any results from this user or AD-Object.

//optional: picture
grafik

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
…ttle typos

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
…mendation for Redis

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
…ut automatic setting of cron mode

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
…x image scaling issues

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
… docs

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
[stable15]  Reduce size of logo and remove background on hover
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
johkoenig and others added 7 commits December 3, 2018 16:28
[stable15] Remove outdated config.php docs in developer manual
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
[stable15] Update required PHP modules
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
[stable15] Fix warnings and links on install page
@blizzz
Copy link
Member

blizzz commented Dec 5, 2018

The stable15 branch reflects the documentation for NC 15, while master is the preparation for the next major release. Thus, stable15 should not be merged into master.

You would want to merge your working branch to master and when it was merged we backport it.

@MorrisJobke
Copy link
Member

@blizzz This looks more like a issue, than a PR.

@victorbw Could you open a new ticket with your comment in here: https://github.com/nextcloud/documentation/issues/new

@MorrisJobke MorrisJobke closed this Dec 5, 2018
@victorbw
Copy link
Author

victorbw commented Dec 5, 2018

moved to #1010

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants