Skip to content

[stable28] enh(settings): Load from disabled users endpoint - #41949

Merged
AndyScherzinger merged 3 commits into
stable28from
backport/stable28/40169
Dec 6, 2023
Merged

[stable28] enh(settings): Load from disabled users endpoint#41949
AndyScherzinger merged 3 commits into
stable28from
backport/stable28/40169

Conversation

@Pytal

@PytalPytal commented Dec 1, 2023

Copy link
Copy Markdown
Member

@PytalPytal added bug 3. to review Waiting for reviews labels Dec 1, 2023
@PytalPytal added this to the Nextcloud 28 milestone Dec 1, 2023
@PytalPytal self-assigned this Dec 1, 2023
@Pytal
Pytal enabled auto-merge December 1, 2023 00:39
Comment threadlib/public/User/Backend/IProvideEnabledStateBackend.php

@nickvergessennickvergessen left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🙈
But didn't check if pagination works properly when you have 100 disabled users and start searching for a user starting with z

@blizzzblizzz mentioned this pull request Dec 1, 2023
5 tasks
@Pytal
Pytalforce-pushed the backport/stable28/40169 branch 2 times, most recently from 9c11e2d to 6a51725CompareDecember 5, 2023 21:58
Pytaland others added 3 commits December 6, 2023 00:08
Signed-off-by: Christopher Ng <chrng8@gmail.com>
(cherry picked from commit 7affbba)
Signed-off-by: Christopher Ng <chrng8@gmail.com>
This matches what was done in the calls and so fixes getting disabled
user list when there are several backends returning disabled users.
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
(cherry picked from commit 85e7887)
@AndyScherzinger
AndyScherzinger merged commit b10f72f into stable28Dec 6, 2023
@AndyScherzinger
AndyScherzinger deleted the backport/stable28/40169 branch December 6, 2023 09:44
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@Pytal@nickvergessen@AndyScherzinger@artonge@come-nc