Skip to content

Conversation

@redblom
Copy link

@redblom redblom commented Jan 27, 2026

  • Resolves: #

Summary

Code moved to contacts app, see nextcloud/contacts#4417

TODO

  • deal with inviteAcceptedDialog code

Checklist

@redblom redblom requested a review from a team as a code owner January 27, 2026 15:43
@redblom redblom requested review from leftybournes, nfebe, salmart-dev and sorbaugh and removed request for a team January 27, 2026 15:43
@redblom redblom marked this pull request as draft January 27, 2026 15:43
@ChristophWurst
Copy link
Member

@redblom why is this moved out of server and could it break OCM when the contacts app is not installed? the app is not shipped. it is optional.

@redblom redblom force-pushed the move-ocm-invite-accepted-route-impl-to-contacts-app branch from 7cb53f3 to 2d2e4a1 Compare January 28, 2026 16:58
@ArtificialOwl
Copy link
Member

@redblom why is this moved out of server and could it break OCM when the contacts app is not installed? the app is not shipped. it is optional.

This is done on my request:

  • it is moved out of server because the implementation of the feature creates its own db table and does not provide an API to manage it,
  • it will not break OCM, it will only not enable the 'invite-accepted' capability if contacts is not installed. the capability is useless without the contacts app,
  • contacts will register the capability and fully manage the db table

@ArtificialOwl
Copy link
Member

let's ping @mickenordin on this one !

@mickenordin mickenordin force-pushed the move-ocm-invite-accepted-route-impl-to-contacts-app branch from 2d2e4a1 to aea3c70 Compare January 28, 2026 18:22
@mickenordin
Copy link
Contributor

let's ping @mickenordin on this one !

I haven't reviewed the change, but in principal I see no problem with it.

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.

4 participants