novuhq / novu

Open-Source Notification Platform. Embeddable Notification Center, E-mail, Push and Slack Integrations.
https://novu.co
Other
35.48k stars 3.92k forks source link

fix(api): error handling #6890

Closed Mayureshd-18 closed 2 days ago

Mayureshd-18 commented 3 weeks ago

What changed? Why was the change needed?

Raw Error was not displayed when caught. Added the error message parameter in the line

Screenshots

Expand for optional sections ### Related enterprise PR ### Special notes for your reviewer
netlify[bot] commented 3 weeks ago

Deploy Preview for novu-stg-vite-dashboard-poc ready!

Name Link
Latest commit 57c350243434840d0f458c6e3fdd20bc5809558f
Latest deploy log https://app.netlify.com/sites/novu-stg-vite-dashboard-poc/deploys/673a7ff503cc140008b24e67
Deploy Preview https://deploy-preview-6890--novu-stg-vite-dashboard-poc.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

Mayureshd-18 commented 1 week ago

@rifont Please review and do the needful.

Thanks

scopsy commented 2 days ago

I'm sorry but I'm not sure what this PR solves as only the e2e file was changed. Could you please submit an issue before to better understand what problem are we trying to solve