Skip to content

Conversation

ArtyomSavchenko
Copy link
Contributor

Screenshot 2025-08-15 at 12 36 15

Signed-off-by: Artem Savchenko <[email protected]>
Signed-off-by: Artem Savchenko <[email protected]>
Signed-off-by: Artem Savchenko <[email protected]>
Copy link
Contributor

@Copilot 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 replaces the channel access control (public/private) with space selection for Telegram channels. Users can now select specific spaces (CardSpace or PersonSpace) where Telegram channels will be synchronized, providing more granular control over where messages appear.

  • Replaced access dropdown with SpaceSelector component allowing users to choose target spaces
  • Updated data structures to store space references instead of access permissions
  • Removed access-related filtering and bulk actions from the UI

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
plugins/telegram-resources/src/components/Configure.svelte Replaced access dropdown with SpaceSelector, updated channel configuration logic and removed access-related UI elements
plugins/telegram-resources/src/api.ts Updated TelegramChannelConfig and TelegramChannelData interfaces to use space references instead of access properties
plugins/telegram-resources/package.json Added @hcengineering/card dependency for CardSpace support

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Signed-off-by: Artem Savchenko <[email protected]>
aonnikov
aonnikov previously approved these changes Aug 15, 2025
Signed-off-by: Artem Savchenko <[email protected]>
@haiodo haiodo merged commit ef2621c into develop Aug 17, 2025
15 checks passed
@haiodo haiodo deleted the fix-space-select branch August 17, 2025 04:17
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.

3 participants