Skip to content

Removed unused imports and linting warnings - #71

Merged
supercell02 merged 1 commit into
mainfrom
Removed-all-linting-warnings
Aug 24, 2026
Merged

Removed unused imports and linting warnings#71
supercell02 merged 1 commit into
mainfrom
Removed-all-linting-warnings

Conversation

@supercell02

Copy link
Copy Markdown
Contributor

This pull request includes minor improvements and dependency updates across several files, as well as a fix to the dependency arrays in custom hooks. The most notable changes are in the hooks for fetching Vapi data, which now correctly include dependencies to prevent stale closures.

Hooks and Dependency Fixes:

  • Updated the dependency array in useVapiAssistants to include getAssistants, ensuring the effect re-runs when the function reference changes.
  • Updated the dependency array in useVapiPhoneNumbers to include getPhoneNumbers, ensuring the effect re-runs when the function reference changes.

Code Cleanup and Minor Updates:

  • Removed an unused import of z from customization-form.tsx for cleaner code.
  • Removed an unused import of Image from dashboard-sidebar.tsx for cleaner code.
  • Added setErrorMessage and setScreen to the dependency array in widget-loading-screen.tsx to ensure correct effect behavior.## What does this change do, and why?

Which area(s) does this touch?

  • Dashboard (apps/web)
  • Chat widget (apps/widget)
  • Embed script (apps/embed)
  • Backend / Convex functions (packages/backend)
  • Shared UI (packages/ui)
  • Dependencies / tooling

How was this tested?

Multi-tenancy / auth impact

Related issues

@vercel

vercel Bot commented Aug 24, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
solvia-web Ready Ready Preview Aug 24, 2026 1:54pm
solvia-widget Ready Ready Preview Aug 24, 2026 1:54pm

Copilot AI lite review requested due to automatic review settings August 24, 2026 13:53

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@supercell02
supercell02 merged commit b2dc97f into main Aug 24, 2026
4 checks passed
@supercell02
supercell02 deleted the Removed-all-linting-warnings branch August 24, 2026 13:55
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.

2 participants