-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Calendar settings #9416
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Calendar settings #9416
Conversation
Signed-off-by: Denis Bykhov <bykhov.denis@gmail.com>
Connected to Huly®: UBERF-12037 |
There was a problem hiding this 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 adds support for primary calendar selection and settings throughout the calendar feature, introduces a formal AccessLevel
enum for calendar and event permissions, and backfills new user
and access
fields on existing calendars via a migration.
- Introduce
AccessLevel
enum and refactor all calendar/event access fields to use it - Add
getPrimaryCalendar
helper and wire it into calendar creation and UI components - Implement a new
CalendarSettings
Svelte component and register it under account settings
Reviewed Changes
Copilot reviewed 37 out of 37 changed files in this pull request and generated 4 comments.
Show a summary per file
File | Description |
---|---|
services/calendar/pod-calendar/src/workspaceClient.ts | Force weekly full sync and simplify event query construction |
services/calendar/pod-calendar/src/sync.ts | Switch to AccessLevel , rename query filters, and set user field on ExternalCalendars |
server-plugins/calendar-resources/src/index.ts | Refactor createCalendar to use AccessLevel , add CalendarSettings integration |
plugins/calendar/src/utils.ts | Add getPrimaryCalendar helper and import core types |
plugins/calendar/src/index.ts | Define PrimaryCalendar interface and AccessLevel enum, register new class IDs |
plugins/calendar-resources/src/components/CalendarSettings.svelte | New UI for managing calendars (visibility, hidden, primary) |
plugins/time-resources/src/utils.ts | Expose findPrimaryCalendar for consuming components |
plugins/time-resources/src/components/PlanView.svelte | Consume findPrimaryCalendar and remove inline ExternalCalendar lookup |
models/calendar/src/migration.ts | Backfill user and access fields on existing calendars |
common/scripts/version.txt | Bump version to 0.7.167 |
Comments suppressed due to low confidence (1)
plugins/time-resources/src/components/PlanView.svelte:28
timeSeparators
is not exported fromutils.ts
, which will cause an import error. Either export it from utils or remove it from this import.
import { findPrimaryCalendar, timeSeparators } from '../utils'
Signed-off-by: Denis Bykhov <bykhov.denis@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tests are failed massive
No description provided.