Skip to content

Conversation

@ArtyomSavchenko
Copy link
Member

@ArtyomSavchenko ArtyomSavchenko commented Oct 7, 2025

Screenshot 2025-10-13 at 12 20 30

Signed-off-by: Artem Savchenko <armisav@gmail.com>
@ArtyomSavchenko ArtyomSavchenko requested a review from aav October 7, 2025 00:27
@huly-github-staging
Copy link

Connected to Huly®: UBERF-13882

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 adds support for telegram reconnection functionality to handle integration disconnection and authorization errors gracefully.

Key changes:

  • Refactored Reconnect.svelte to reuse Connect.svelte with reconnect flag
  • Added error handling for unauthorized errors in integration state management
  • Implemented proper disconnected state visualization in integration cards

Reviewed Changes

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

Show a summary per file
File Description
plugins/telegram-resources/src/components/Reconnect.svelte Simplified to reuse Connect component with reconnect=true
plugins/telegram-resources/src/components/Connect.svelte Enhanced with reconnect functionality and improved error handling
plugins/telegram-resources/src/components/IntegrationState.svelte Added unauthorized error detection and handling
plugins/telegram-resources/src/api.ts Added retry configuration for HTTP requests
plugins/setting-resources/src/components/integrations/IntegrationLabel.svelte Added disconnected state visualization
plugins/setting-resources/src/components/integrations/IntegrationCard.svelte Added reconnect button for disconnected integrations
packages/integration-client/src/utils.ts Added utility functions for disabled state and unauthorized error detection
packages/integration-client/src/types.ts Extended IntegrationClient interface with new methods
packages/integration-client/src/client.ts Implemented removeSecrets and setIntegrationEnabled methods
Multiple language files Added "Disconnected" translation strings
packages/theme/styles/_colors.scss Added orange color theme for disconnected state

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

Signed-off-by: Artem Savchenko <armisav@gmail.com>
Signed-off-by: Artem Savchenko <armisav@gmail.com>
Signed-off-by: Artem Savchenko <armisav@gmail.com>
Signed-off-by: Artem Savchenko <armisav@gmail.com>
Signed-off-by: Artem Savchenko <armisav@gmail.com>
Signed-off-by: Artem Savchenko <armisav@gmail.com>
@ArtyomSavchenko ArtyomSavchenko marked this pull request as ready for review October 13, 2025 05:55
@ArtyomSavchenko ArtyomSavchenko merged commit 5182f6b into develop Oct 13, 2025
15 checks passed
@BykhovDenis BykhovDenis deleted the telegram-reconnect branch October 22, 2025 18:25
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