Skip to content

chore: remove HubSpot chat integration#856

Merged
tannerlinsley merged 1 commit intomainfrom
taren/lucid-feistel-bb7521
Apr 25, 2026
Merged

chore: remove HubSpot chat integration#856
tannerlinsley merged 1 commit intomainfrom
taren/lucid-feistel-bb7521

Conversation

@tannerlinsley
Copy link
Copy Markdown
Member

@tannerlinsley tannerlinsley commented Apr 25, 2026

Summary

  • Removes the HubSpot chat widget integration (hook, root call, CSS rule)
  • Deletes the now-unused useScript hook (was only consumed by useHubSpotChat)

Test plan

  • tsc clean
  • lint clean

Summary by CodeRabbit

  • Revert
    • Removed HubSpot chat widget integration from the application, including chat loading logic, script injection, and related styling. The chat widget will no longer be displayed to end-users.

@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 25, 2026

Deploy Preview for tanstack ready!

Name Link
🔨 Latest commit 271fe3c
🔍 Latest deploy log https://app.netlify.com/projects/tanstack/deploys/69ec23b27003550007032a84
😎 Deploy Preview https://deploy-preview-856--tanstack.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 32 (🔴 down 27 from production)
Accessibility: 90 (no change from production)
Best Practices: 83 (🔴 down 9 from production)
SEO: 97 (no change from production)
PWA: 70 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@tannerlinsley tannerlinsley merged commit 4e8559e into main Apr 25, 2026
6 of 8 checks passed
@tannerlinsley tannerlinsley deleted the taren/lucid-feistel-bb7521 branch April 25, 2026 02:15
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 25, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 22de37f8-6eac-4eca-bef0-430351c6196b

📥 Commits

Reviewing files that changed from the base of the PR and between 180ddd3 and 271fe3c.

📒 Files selected for processing (4)
  • src/hooks/useHubSpotChat.ts
  • src/hooks/useScript.ts
  • src/routes/__root.tsx
  • src/styles/app.css

📝 Walkthrough

Walkthrough

This pull request removes all HubSpot chat widget functionality from the codebase, including the custom hooks that managed conditional script loading and the associated styling. The integration is completely eliminated across hooks, routing, and stylesheets.

Changes

Cohort / File(s) Summary
HubSpot Chat Integration Removal
src/hooks/useHubSpotChat.ts, src/hooks/useScript.ts
Deleted the useHubSpotChat hook, HubSpotChatConfig interface, and the underlying useScript hook that handled dynamic script injection with configurable delays.
HubSpot Chat Runtime References
src/routes/__root.tsx
Removed import and invocation of the useHubSpotChat hook from the root shell component.
HubSpot Chat Styling
src/styles/app.css
Removed CSS rules that applied positional translation and dark-mode styling to the HubSpot messages iframe container.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Poem

🐰 The chatbot hops away, no more,
HubSpot scripts deleted to the core,
Styles erased with a whisker's twitch,
No iframe left without a glitch!
Clean codebase hops forevermore. 🎉

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch taren/lucid-feistel-bb7521

Comment @coderabbitai help to get the list of available commands and usage tips.

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