Skip to content

fix(integrations): fix icons, redirects, and empty expandables#45

Open
janelletam wants to merge 4 commits into
masterfrom
janelle/docs-116-integrations-fix-inconsistencies-part-1
Open

fix(integrations): fix icons, redirects, and empty expandables#45
janelletam wants to merge 4 commits into
masterfrom
janelle/docs-116-integrations-fix-inconsistencies-part-1

Conversation

@janelletam

Copy link
Copy Markdown
Contributor
image image

@linear-code

linear-code Bot commented Jun 15, 2026

Copy link
Copy Markdown

DOCS-116

@greptile-apps

greptile-apps Bot commented Jun 15, 2026

Copy link
Copy Markdown

Greptile Summary

This PR fixes three categories of issues in the integrations documentation: brand icons for WhatsApp, Telegram, and Messenger are switched from generic Lucide icons to real simple-icons equivalents (with the icon bundle updated accordingly); several new redirects are added and the existing /tutorial redirect is corrected to include the baseUrl prefix; and empty <Expandable> sections in SchemaFields.tsx are suppressed by adding Object.keys().length > 0 guards.

  • astro.config.mjs: Adds five redirects (all using ${baseUrl} for relative paths, one external), fixes the pre-existing /tutorial redirect, and registers whatsapp, telegram, messenger in the simple-icons bundle.
  • SchemaFields.tsx: Guards both the object and array expandable rendering paths against empty property maps, preventing hollow collapsible UI elements.
  • introduction.mdx: Updates card icons to brand icons, updates the WhatsApp href to the /introduction subpage, and capitalizes the page title.

Confidence Score: 4/5

Safe to merge after confirming the /desk external URL is correct and verifying Telegram/Messenger card links reach their intended pages.

The SchemaFields fix is straightforward and correct. The redirect and icon changes are low-risk, but the /desk external URL (https://desk.support.channel/) is unusual and could silently misdirect users if it's a typo, and the Telegram/Messenger hrefs may point to non-existent routes if those guides have the same structure as WhatsApp.

astro.config.mjs (the /desk redirect destination) and introduction.mdx (Telegram/Messenger card hrefs)

Important Files Changed

Filename Overview
astro.config.mjs Fixes /tutorial redirect to include baseUrl, adds five new redirects, and expands simple-icons bundle to include whatsapp, telegram, and messenger; the /desk external URL warrants a quick double-check
src/components/integrations/SchemaFields.tsx Adds Object.keys().length > 0 guards for both object and array expandables to prevent rendering empty collapsible sections; fix is correct and safe
src/content/docs/integrations/get-started/introduction.mdx Switches card icons to real brand icons (simple-icons), updates WhatsApp href to /introduction subpage, and capitalizes page title; Telegram and Messenger hrefs keep trailing-slash paths unlike WhatsApp

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["/tutorial"] -->|redirect| B["baseUrl/tutorial/introduction"]
    C["/integrations"] -->|redirect| D["baseUrl/integrations/get-started/introduction"]
    E["/integrations/integration-guides/whatsapp"] -->|redirect| F["baseUrl/.../whatsapp/introduction"]
    G["/integrations/integration-guides/hitl"] -->|redirect| H["baseUrl/.../hitl/introduction"]
    I["/studio"] -->|redirect| J["baseUrl/studio/introduction"]
    K["/desk"] -->|redirect| L["https://desk.support.channel/"]

    style L fill:#ffe0b2,stroke:#f57c00
    style K fill:#ffe0b2,stroke:#f57c00
Loading

Reviews (1): Last reviewed commit: "fix(integrations): hide empty expandable..." | Re-trigger Greptile

Comment thread astro.config.mjs
Comment thread src/content/docs/integrations/get-started/introduction.mdx
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.

1 participant