Skip to content

feat: Add back searching in disabled user list#45370

Merged
come-nc merged 4 commits intomasterfrom
feat/add-back-search-in-disabled-users
May 23, 2024
Merged

feat: Add back searching in disabled user list#45370
come-nc merged 4 commits intomasterfrom
feat/add-back-search-in-disabled-users

Conversation

@come-nc
Copy link
Contributor

@come-nc come-nc commented May 16, 2024 •
edited
Loading

Summary

When disabled users where moved to their own endpoint we overlooked search, so adding it back.
The search is done case-insensitive in uid and display name.

TODO

  • Decide if we should also search in email field, and implement it
  • Decide if the BC break is ok or should be avoided with a new interface

Checklist

@come-nc come-nc added the 2. developing Work in progress label May 16, 2024
@come-nc come-nc added this to the Nextcloud 30 milestone May 16, 2024
@come-nc come-nc requested review from Pytal and artonge May 16, 2024 14:18
@come-nc come-nc self-assigned this May 16, 2024
@come-nc
Copy link
Contributor Author

come-nc commented May 16, 2024 •
edited
Loading

TODO

  • Decide if we should also search in email field, and implement it

Implemented.

  • Decide if the BC break is ok or should be avoided with a new interface

I think this is fine, because a class implementing the interface for 30 also implements the one for 28.0.0, as it’s only a new parameter.
So, as the breaking is minimal and it’s easy for concerned apps (and that’s only apps implementing a user backend with special disabled user treatment, so only user_ldap to my knowlegde) to support all Nextcloud versions.

@come-nc come-nc added 3. to review Waiting for reviews feature: users and groups and removed 2. developing Work in progress labels May 16, 2024
@come-nc
Copy link
Contributor Author

come-nc commented May 16, 2024

/compile /

@come-nc come-nc requested a review from provokateurin as a code owner May 16, 2024 15:04
provokateurin
provokateurin reviewed May 16, 2024
View reviewed changes
Copy link
Member

@provokateurin provokateurin left a comment

Choose a reason for hiding this comment

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

I would prefer separate methods for searching quite a lot over changing the behavior of the existing methods. Would that be possible or is it too much work or impractical?

@provokateurin
Copy link
Member

provokateurin commented May 16, 2024

Ok in light of the discussion in #44936 it makes sense to extend the existing endpoint instead of creating a new one

artonge
artonge approved these changes May 16, 2024
View reviewed changes
Copy link
Contributor

@artonge artonge left a comment •
edited
Loading

Choose a reason for hiding this comment

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

So BC is not a real BC?

Edit: seen your second comment

Pytal
Pytal requested changes May 16, 2024
View reviewed changes
come-nc added 2 commits May 23, 2024 09:08
@come-nc
feat: Add back searching in disabled user list
6d56f35 
When disabled users where moved to their own endpoint we overlooked
 search, so adding it back.
The search is done case-insensitive in uid and display name.

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
@come-nc
fix: Also search in email field for disabled users
8044edf 
To match what is done for Database backend for enabled users

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
@come-nc come-nc force-pushed the feat/add-back-search-in-disabled-users branch from bfde1f7 to 0505a1f Compare May 23, 2024 07:10
@come-nc
Copy link
Contributor Author

come-nc commented May 23, 2024

/compile /

come-nc and others added 2 commits May 23, 2024 10:42
@come-nc
fix: Update openapi spec for provisioning_api app
96b6662 
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
@nextcloud-command @come-nc
chore(assets): Recompile assets
9a523e9 
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@come-nc come-nc force-pushed the feat/add-back-search-in-disabled-users branch from e5c0dbb to 9a523e9 Compare May 23, 2024 08:42
@come-nc come-nc merged commit 2fa099a into master May 23, 2024
@come-nc come-nc deleted the feat/add-back-search-in-disabled-users branch May 23, 2024 12:51
@come-nc
Copy link
Contributor Author

come-nc commented May 23, 2024

/backport to stable29

@come-nc
Copy link
Contributor Author

come-nc commented May 23, 2024

/backport to stable28

@backportbot backportbot bot mentioned this pull request May 23, 2024
Merged
2 tasks
@backportbot backportbot bot mentioned this pull request May 23, 2024
Merged
2 tasks
@nickvergessen
Copy link
Member

nickvergessen commented Jul 18, 2024 •
edited
Loading

@come-nc Please document in https://docs.nextcloud.com/server/latest/developer_manual/app_publishing_maintenance/app_upgrade_guide/upgrade_to_30.html#id2

@nickvergessen nickvergessen added the pending documentation This pull request needs an associated documentation update label Jul 18, 2024
@blizzz blizzz mentioned this pull request Jul 24, 2024
Merged
@come-nc come-nc mentioned this pull request Apr 3, 2025
Merged
@come-nc come-nc removed the pending documentation This pull request needs an associated documentation update label Apr 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@provokateurin provokateurin provokateurin left review comments

@Altahrim Altahrim Altahrim approved these changes

@artonge artonge artonge approved these changes

@Pytal Pytal Pytal approved these changes

Assignees

@come-nc come-nc

Labels

3. to review Waiting for reviews feature: users and groups

Projects

None yet

Milestone

Nextcloud 30

Development

Successfully merging this pull request may close these issues.

[Bug]: Search in the"disabled users" section does not work

7 participants

@come-nc @provokateurin @nickvergessen @Altahrim @artonge @Pytal @nextcloud-command