Skip to content

feat: implement user mentions in Collabora docs#2601

Open
JammingBen wants to merge 2 commits into
mainfrom
feat/collabora-mentions
Open

feat: implement user mentions in Collabora docs#2601
JammingBen wants to merge 2 commits into
mainfrom
feat/collabora-mentions

Conversation

@JammingBen
Copy link
Copy Markdown
Member

@JammingBen JammingBen commented May 27, 2026

Users can be mentioned with @ in Collabora documents, triggering a notification to the mentioned user. Also refactors all Collabora postMessage stuff and the task for loading shares to dedicated composables.

Note this this is still missing the actual notification parts from the server. But it doesn't hurt to have the functionality in Web before that.

@JammingBen JammingBen self-assigned this May 27, 2026
@JammingBen JammingBen force-pushed the feat/collabora-mentions branch 4 times, most recently from 7711716 to 0f9fdd8 Compare May 28, 2026 08:41
@JammingBen JammingBen marked this pull request as ready for review May 28, 2026 08:46
Comment on lines +302 to +306
await httpAuthenticated.post(urlJoin(configStore.serverUrl, 'collaboration/notify'), {
fileID: unref(resource).fileId,
userIDs: unref(userIdsToMention),
type: 'mention'
})
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@fschade JFYI this is the notification POST request I'm sending to the server as we discussed.

Users can be mentioned with `@` in Collabora documents, triggering a
notification to the mentioned user.

Also refactors all Collabora postMessage stuff to a dedicated
composable.
@JammingBen JammingBen force-pushed the feat/collabora-mentions branch from 0f9fdd8 to 4f40d1d Compare May 28, 2026 09:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant