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

[stable11] some sharing settings #5215

Merged
merged 6 commits into from Jul 5, 2017

Conversation

Projects
None yet
3 participants
@schiessle
Member

schiessle commented Jun 1, 2017

backport of #3567

cc @blizzz

schiessle and others added some commits Feb 21, 2017

allow to configure a min-length of search strings for auto-compeltion…
… and a max number for of results returned

Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>
Improve the UX for sharing settings
* shows a info when list is potentially truncated
* shows a warning when characters length is not enough

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Use intval() for validation of config options
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
properly include class
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
use (int) instead of intval for performance reasons
Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>
fix typo
Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>

@schiessle schiessle added this to the Nextcloud 11.0.4 milestone Jun 1, 2017

@schiessle schiessle requested review from nickvergessen and blizzz Jun 1, 2017

@blizzz

blizzz approved these changes Jun 1, 2017

works for me

@MorrisJobke

This comment has been minimized.

Show comment
Hide comment
@MorrisJobke

MorrisJobke Jun 15, 2017

Member

@schiessle Any reason to backport this to stable11?

Member

MorrisJobke commented Jun 15, 2017

@schiessle Any reason to backport this to stable11?

@MorrisJobke MorrisJobke merged commit bb1ff50 into stable11 Jul 5, 2017

1 of 2 checks passed

continuous-integration/drone/push the build failed
Details
continuous-integration/drone/pr the build was successful
Details

@MorrisJobke MorrisJobke deleted the autocomplete-settings-stable11 branch Jul 5, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment