Skip to content

feat: Add Talk meeting button to calendar entries - #22

Open
kdanielm wants to merge 2 commits into
bulwarkmail:mainfrom
kdanielm:main
Open

kdanielm wants to merge 2 commits into
bulwarkmail:mainfrom
kdanielm:main

Conversation

@kdanielm

Copy link
Copy Markdown

Summary

This adds the ability to add a Nextcloud Talk meeting link to a calendar entry similar to what the Jitsi plugin does

Changes

  • Added permissions to manifest
    • calendar:read
    • calendar:write
    • ui:calendar-action
  • Added setting enableTalkMeetingsButton in manifest
  • Added translations for en and de (Sorry, I unfortunately can't translate the rest) for the identifiers:
    • setting.enableTalkMeetingsButton
    • setting.enableTalkMeetingsButtonHelp
    • talkMeeting.add
    • talkMeeting.added
    • talkMeeting.error
  • Added createTalkRoom method to client (client.js)
  • Added talkmeeting-button.js, which contains the button and the click logic

Related issues

Closes #

Type of change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Refactor / code quality improvement
  • Chore / dependency update / CI change

Checklist

  • I have read the Contributing Guide
  • My code follows the project's code style and conventions
  • I have run npm run typecheck && npm run lint and there are no errors
  • The build passes (npm run build)
  • I have tested my changes locally
  • I have added or updated documentation if needed
  • I have updated translations (locales/) if my changes affect user-facing text
  • I have included screenshots or a screen recording for UI changes

Screenshots / demo

Added settings option:
image

Add Talk meeting button in event ui:
image

Notes for reviewers

@kdanielm
kdanielm marked this pull request as ready for review October 19, 2026 13:16

This branch has not been deployed

No deployments
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.

1 participant