decrease initial users to load to 50

[butonic]

Prevents timeouts on the initial loading of users. proper fix will be in #10994
Workaround for #24734

@DeepDiver1975 @PVince81 Should be backported to 9.0.3

[mention-bot]

By analyzing the blame information on this pull request, we identified @ringmaster, @MorrisJobke and @PVince81 to be potential reviewers

[PVince81]

As discussed 👍

[PVince81]

@nickvergessen second review ?

[nickvergessen]

👍 I hope we have no JS test that checks this value 😛

[guruz]

Would be cool if commit subject lines had an identifying prefix.
In the client we do things like "SyncEngine: fix mem leak" or "Toolbar: fix mem leak" or whatever. Then it's easier to identify what this is about.
Here I thought it was an LDAP issue or so.

[nickvergessen]

Well we use the light blue and purple labels for that ;)

[DeepDiver1975]

Nice!

[DeepDiver1975]

@DeepDiver1975 @PVince81 Should be backported to 9.0.3

please submit backport pr - thx

[DeepDiver1975]

@PVince81 if I remember correctly we had issues in the past with the scrolling functionality if the initial set was to small. Has this been addressed / tested?

[PVince81]

@DeepDiver1975 I haven't retested this, but note that we already had the small value in all past versions. I believe @butonic increased it to improve UX and avoid having the user scroll, wait, scroll, wait too often.

[georgehrke]

👍

[butonic]

I increased the initial number of users to 250 because we saw a complaint about having to scroll to reach a user ... well there is user search anyway. maybe increasing subsequent user fetches from 10 to 30 eases the pain a little?

[PVince81]

stable9: #25105

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.