chore: remove Intercom integration and chat support components#8875
chore: remove Intercom integration and chat support components#8875sriramveeraghanta merged 1 commit intopreviewfrom
Conversation
Intercom is no longer used. This removes all related frontend components, hooks, custom events, API config, types, and i18n keys.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (28)
💤 Files with no reviewable changes (26)
📝 WalkthroughWalkthroughThis PR removes Intercom chat support functionality across the application. Changes include eliminating the Intercom configuration UI component from the admin dashboard, removing Intercom-related configuration from API endpoints and configuration variables, deleting chat support hooks and event handlers from the web frontend, and removing related translation strings from 23 supported locales. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
This PR removes the Intercom-based chat support integration across the monorepo (frontend UI, admin settings, API instance config surface, shared types, and locale strings), leaving telemetry as the remaining related admin setting.
Changes:
- Removed chat support UI entry points (sidebar help item + Power-K command) and the supporting hook/custom event.
- Removed Intercom configuration from the API instance endpoint and instance config variable registry, plus the related shared instance types.
- Cleaned up now-orphaned i18n strings across all locales and renamed the admin section header to “Telemetry”.
Reviewed changes
Copilot reviewed 28 out of 28 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| packages/types/src/instance/base.ts | Removes Intercom fields and configuration key type from shared instance config types. |
| packages/i18n/src/locales/zh-TW/translations.ts | Removes message_support translation key. |
| packages/i18n/src/locales/zh-CN/translations.ts | Removes message_support translation key. |
| packages/i18n/src/locales/vi-VN/translations.ts | Removes message_support translation key. |
| packages/i18n/src/locales/ua/translations.ts | Removes message_support and power_k.help_actions.chat_with_us translation keys. |
| packages/i18n/src/locales/tr-TR/translations.ts | Removes message_support translation key. |
| packages/i18n/src/locales/sk/translations.ts | Removes message_support translation key. |
| packages/i18n/src/locales/ru/translations.ts | Removes message_support and power_k.help_actions.chat_with_us translation keys. |
| packages/i18n/src/locales/ro/translations.ts | Removes message_support translation key. |
| packages/i18n/src/locales/pt-BR/translations.ts | Removes message_support translation key. |
| packages/i18n/src/locales/pl/translations.ts | Removes message_support translation key. |
| packages/i18n/src/locales/ko/translations.ts | Removes message_support translation key. |
| packages/i18n/src/locales/ja/translations.ts | Removes message_support translation key. |
| packages/i18n/src/locales/it/translations.ts | Removes message_support translation key. |
| packages/i18n/src/locales/id/translations.ts | Removes message_support translation key. |
| packages/i18n/src/locales/fr/translations.ts | Removes message_support translation key. |
| packages/i18n/src/locales/es/translations.ts | Removes message_support translation key. |
| packages/i18n/src/locales/en/translations.ts | Removes message_support and power_k.help_actions.chat_with_us translation keys. |
| packages/i18n/src/locales/de/translations.ts | Removes message_support translation key. |
| packages/i18n/src/locales/cs/translations.ts | Removes message_support translation key. |
| apps/web/core/hooks/use-chat-support.ts | Deletes the hook used to open Intercom chat support. |
| apps/web/core/custom-events/chat-support.ts | Deletes the custom event wrapper used by the chat support hook. |
| apps/web/core/components/workspace/sidebar/help-section/root.tsx | Removes “Message support” help menu item and related dependencies. |
| apps/web/core/components/power-k/config/help-commands.ts | Removes the “Chat with us” Power-K help command. |
| apps/api/plane/utils/instance_config_variables/core.py | Removes Intercom env-backed config variables from the core config variable list. |
| apps/api/plane/license/api/views/instance.py | Stops returning is_intercom_enabled and intercom_app_id from /api/instances/ config. |
| apps/admin/app/(all)/(dashboard)/general/intercom.tsx | Deletes the admin UI component for Intercom/chat support configuration. |
| apps/admin/app/(all)/(dashboard)/general/form.tsx | Removes Intercom config wiring and renames the section header to “Telemetry”. |
Summary
IntercomConfig,useChatSupporthook,ChatSupportEventcustom event)is_intercom_enabled,intercom_app_idfrom instance typesmessage_support,chat_with_us) across all 19 localesTest plan
/api/instances/endpoint no longer returnsis_intercom_enabledorintercom_app_idSummary by CodeRabbit