Skip to content

Comments

feat(teams): support federated users/groups search when adding team members#5037

Merged
cristianscheid merged 3 commits intomainfrom
feat/2303/contacts-allow-federated-team-member-search
Feb 17, 2026
Merged

feat(teams): support federated users/groups search when adding team members#5037
cristianscheid merged 3 commits intomainfrom
feat/2303/contacts-allow-federated-team-member-search

Conversation

@cristianscheid
Copy link
Contributor

@cristianscheid cristianscheid commented Jan 24, 2026

Summary

Adds possibility to search for federated users (accounts from other instances) and federated groups when adding members to a team in the Contacts app.

Context

Changes

  • Enabled federated users and groups in CIRCLES_MEMBER_GROUPING by uncommenting the previously disabled entries in src/models/constants.ts
  • Fixed avatar rendering for federated entities: Modified src/components/EntityPicker/EntitySearchResult.vue and src/components/EntityPicker/EntityBubble.vue to conditionally pass the user prop to NcUserBubble only for ShareType.User entities
    • Federated users triggered 404 errors when the component attempted to load avatars from non-existent local paths
    • For non-user share types (federated users, groups, emails), the user prop is set to undefined, causing NcUserBubble to display initials from displayName instead of fetching an avatar
    • This approach was inspired by how it's done on apps/files_sharing/src/components/SharingInput.vue for handling different share types in the Files app external sharing feature

Before

before.webm

After

after.webm

Note

When testing locally, the federated search functionality works correctly (remote users appear in search results). However, when attempting to actually add the remote member to a team, an error occurs indicating the remote instance is unreachable. I believe this is expected behavior when running locally, as a similar error occurs when trying to add an external share in the Files app using the same remote account, which is the reference implementation mentioned in the issue and was not modified in this PR.

Checklist

@codecov
Copy link

codecov bot commented Jan 24, 2026

@cristianscheid cristianscheid force-pushed the feat/2303/contacts-allow-federated-team-member-search branch from 6a0a354 to 96d743d Compare January 24, 2026 12:41
@hamza221 hamza221 added the enhancement New feature or request label Jan 24, 2026
@cristianscheid cristianscheid self-assigned this Jan 26, 2026
@cristianscheid cristianscheid force-pushed the feat/2303/contacts-allow-federated-team-member-search branch from 96d743d to fa7bad4 Compare February 4, 2026 11:21
@hamza221
Copy link
Contributor

hamza221 commented Feb 4, 2026

/backport to stable33

We don't backport feat commits because of conventional app release, Either change the commit to a fix or we can release 8.4.0 once merged.

And contacts app versions are not same as server check here for compatibility with server https://apps.nextcloud.com/apps/contacts

@ArtificialOwl
Copy link
Member

we should have a 8.4.0 for 33 only instead of backporting this to 8.3.x

@hamza221
Copy link
Contributor

hamza221 commented Feb 4, 2026

we should have a 8.4.0 for 33 only instead of backporting this to 8.3.x

this is also doable

@cristianscheid cristianscheid force-pushed the feat/2303/contacts-allow-federated-team-member-search branch 2 times, most recently from 780c229 to 3d0b39c Compare February 4, 2026 12:56
@github-actions
Copy link

github-actions bot commented Feb 8, 2026

Hello there,
Thank you so much for taking the time and effort to create a pull request to our Nextcloud project.

We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process.

Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6

Thank you for contributing to Nextcloud and we hope to hear from you soon!

(If you believe you should not receive this message, you can add yourself to the blocklist.)

Signed-off-by: Cristian Scheid <cristianscheid@gmail.com>
@cristianscheid cristianscheid force-pushed the feat/2303/contacts-allow-federated-team-member-search branch 2 times, most recently from 17879d2 to 74bbff3 Compare February 9, 2026 17:38
…ng for members to add

Signed-off-by: Cristian Scheid <cristianscheid@gmail.com>
@cristianscheid cristianscheid force-pushed the feat/2303/contacts-allow-federated-team-member-search branch from 74bbff3 to e8f3acb Compare February 9, 2026 17:44
… enabled for the team

Signed-off-by: Cristian Scheid <cristianscheid@gmail.com>
@cristianscheid cristianscheid force-pushed the feat/2303/contacts-allow-federated-team-member-search branch from 8391f8f to e15dfc8 Compare February 16, 2026 22:49
@cristianscheid cristianscheid merged commit 7200847 into main Feb 17, 2026
67 of 68 checks passed
@cristianscheid cristianscheid deleted the feat/2303/contacts-allow-federated-team-member-search branch February 17, 2026 11:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews enhancement New feature or request feedback-requested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants