Skip to content

feat(UserPlugin): Include teams in group search#58603

Open
pymnh wants to merge 2 commits intonextcloud:masterfrom
pymnh:feat/extend-group-search-to-teams
Open

feat(UserPlugin): Include teams in group search#58603
pymnh wants to merge 2 commits intonextcloud:masterfrom
pymnh:feat/extend-group-search-to-teams

Conversation

@pymnh
Copy link

@pymnh pymnh commented Feb 26, 2026

Summary

  • Extend UserPlugin: If autocompletion is enabled for groups and the teams interface is available, also search in teams.

Changes:

  • Update the private getTeamsForUser() function to return a dict (user id => display name) instead of only user id
  • Add the updated getTeamsForUser() and the preexisting hasTeamSupport() to the public ITeamManager class
  • Previously there was only one usage of the private getTeamsForUser() -> Adjust it to the new return value (dict instead of array), use the public ITeamInterface instead of TeamInterface and adjust the mocks for testing.

TODO

  • There does not seem to be unit tests for UserPlugin. I am not sure if I should add one to test the behaviour, if so I would need a bit of guidance

Checklist

AI

I used AI to adjust the test mocks and comments 🙂

@pymnh pymnh force-pushed the feat/extend-group-search-to-teams branch from 9f5358a to 403a7be Compare February 26, 2026 23:38
@pymnh pymnh marked this pull request as ready for review February 26, 2026 23:39
@pymnh pymnh requested a review from a team as a code owner February 26, 2026 23:39
@pymnh pymnh requested review from Altahrim, leftybournes, nfebe and salmart-dev and removed request for a team February 26, 2026 23:39
@pymnh
Copy link
Author

pymnh commented Feb 26, 2026

Hi @mejo- , maybe you can have a look at my changes, since this also addresses the usability of the collectives app?

Signed-off-by: Peymaneh <peymaneh@posteo.net>
@pymnh pymnh force-pushed the feat/extend-group-search-to-teams branch from 403a7be to 4713684 Compare February 26, 2026 23:48
@pymnh
Copy link
Author

pymnh commented Feb 27, 2026

Had some trouble setting up cs-fixer and rector with latest Fedora (PHP 8.4), should pass the CI now, I believe. 🙈

Signed-off-by: Peymaneh <peymaneh@posteo.net>
@pymnh pymnh force-pushed the feat/extend-group-search-to-teams branch from 8e0d92f to 6a9c3fc Compare February 27, 2026 21:06
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.

Autocompletion for fellow team members and known users

1 participant