Skip to content

Conversation

@AlexAndBear
Copy link
Contributor

Description

Related Issue

How Has This Been Tested?

  • test environment:
  • test case 1:
  • test case 2:
  • ...

Types of changes

  • Bugfix
  • Enhancement (a change that doesn't break existing code or deployments)
  • Breaking change (a modification that affects current functionality)
  • Technical debt (addressing code that needs refactoring or improvements)
  • Tests (adding or improving tests)
  • Documentation (updates or additions to documentation)
  • Maintenance (like dependency updates or tooling adjustments)

@AlexAndBear AlexAndBear requested a review from kulmann November 21, 2025 13:09
@AlexAndBear AlexAndBear self-assigned this Nov 21, 2025
@AlexAndBear AlexAndBear marked this pull request as ready for review November 21, 2025 15:16
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR implements conditional rendering of calendar-related UI elements based on the FRONTEND_DISABLE_RADICALE environment variable. When this variable is set to true, the calendar menu item and account calendar navigation are hidden from the user interface.

Key Changes:

  • Added support-radicale capability to control calendar feature visibility
  • Conditionally rendered calendar navigation items and app menu extensions based on the capability
  • Updated OpenCloud commit ID reference

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
packages/web-runtime/src/pages/account/accountLayout.vue Modified to conditionally render calendar navigation item based on supportRadicale capability
packages/web-runtime/src/extensions.ts Updated to conditionally include calendar app menu extension when radicale is supported
packages/web-pkg/src/composables/piniaStores/capabilities.ts Added supportRadicale computed property to expose the radicale support capability
packages/web-client/src/ocs/capabilities.ts Added support-radicale boolean property to the core capabilities interface
.woodpecker.env Updated OpenCloud commit ID to latest version

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

@kulmann kulmann left a comment

Choose a reason for hiding this comment

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

Code looks good 👍

@kulmann kulmann merged commit 470caa4 into main Nov 24, 2025
28 checks passed
@kulmann kulmann deleted the issues/1636 branch November 24, 2025 11:04
openclouders pushed a commit that referenced this pull request Nov 24, 2025
…rue (#1660)

* fix: hide calendar when env var FRONTEND_DISABLE_RADICALE is set to true
@openclouders openclouders mentioned this pull request Nov 24, 2025
1 task
AlexAndBear added a commit that referenced this pull request Nov 24, 2025
…rue (#1660)

* fix: hide calendar when env var FRONTEND_DISABLE_RADICALE is set to true

(cherry picked from commit 470caa4)
@AlexAndBear
Copy link
Contributor Author

💚 All backports created successfully

Status Branch Result
stable-4.2

Questions ?

Please refer to the Backport tool documentation

AlexAndBear added a commit that referenced this pull request Nov 24, 2025
…rue (#1660) (#1671)

* fix: hide calendar when env var FRONTEND_DISABLE_RADICALE is set to true

(cherry picked from commit 470caa4)
@openclouders openclouders mentioned this pull request Nov 21, 2025
1 task
@openclouders openclouders mentioned this pull request Dec 15, 2025
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

App Switcher: Show Calendar only if its enabled

3 participants