inngest / inngest-js

The developer platform for easily building reliable workflows with zero infrastructure for TypeScript & JavaScript
https://www.inngest.com/
GNU General Public License v3.0
395 stars 40 forks source link

`Inngest#send()` error messages are vague #94

Closed jpwilliams closed 1 year ago

jpwilliams commented 1 year ago

Summary

We have a list of errors thrown by Inngest#send() to inform the user that Inngest did not successfully receive their event payload. See src/components/Inngest.ts#L151.

None of the errors provides helpful information, requiring a user to guess what's wrong.

Let's adjust so that every error message tells a user:

djfarrelly commented 1 year ago

Yeah, this is remanent of the original SDK release where the Inngest Event API was not returning any response except for the status code. I believe we have better coverage there, but if we return better errors from the Event API both in open source and in Cloud, we can remove this logic and pass through issues!

jpwilliams commented 1 year ago

This work has stated and will continue with #180.