search

novuhq / novu

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

fix(api): Remove double logging of Bridge errors #6864

Closed SokratisVidros closed 2 weeks ago

SokratisVidros commented 2 weeks ago

What changed? Why was the change needed?

We log once in ExecuteBridgeRequest, so there is no need to log the error again in get-bridge-status.usecase.ts

Screenshots

The fix should remove the log entry in the red-dotted box

Screenshot 2024-11-05 at 23 34 27
netlify[bot] commented 2 weeks ago

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

Name Link
Latest commit e8faf2911624d197af7861ec3a8c55c661d39189
Latest deploy log https://app.netlify.com/sites/novu-stg-vite-dashboard-poc/deploys/672a8f9b17e19c00085b3583
Deploy Preview https://deploy-preview-6864--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.