cal.com icon indicating copy to clipboard operation
cal.com copied to clipboard

feat: Improve onError Handling in Atoms: Surface API Error Messages for Toast Notifications

Open weknowyourgame opened this issue 7 months ago • 2 comments

Mandatory Tasks (DO NOT REMOVE)

  • [x] I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • [x] I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. If N/A, write N/A here and check the checkbox.
  • [x] I confirm automated tests are in place that prove my fix is effective or that my feature works.

Summary by cubic

Improved error handling in Atoms to show detailed API error messages in toast notifications instead of generic errors.

  • Bug Fixes
    • Updated all onError handlers to use a new getErrorMessage utility for clearer user feedback.

weknowyourgame avatar Jun 13 '25 06:06 weknowyourgame

@weknowyourgame is attempting to deploy a commit to the cal Team on Vercel.

A member of the Team first needs to authorize it.

vercel[bot] avatar Jun 13 '25 06:06 vercel[bot]

Graphite Automations

"Add consumer team as reviewer" took an action on this PR • (06/13/25)

1 reviewer was added to this PR based on Keith Williams's automation.

"Add community label" took an action on this PR • (06/13/25)

1 label was added to this PR based on Keith Williams's automation.

graphite-app[bot] avatar Jun 13 '25 06:06 graphite-app[bot]

Hey @SomayChauhan from what i understood from the review I implemented a global interceptor handle API error toasts. Please let me know what you think.

weknowyourgame avatar Jun 17 '25 18:06 weknowyourgame

This PR is being marked as stale due to inactivity.

github-actions[bot] avatar Jul 02 '25 00:07 github-actions[bot]