NangoHQ / nango

A single API for all your integrations.
https://www.nango.dev
Other
4.33k stars 382 forks source link

fix(slack-notifications): [nan-1164] dedupe slack notifications #2421

Closed khaliqgant closed 5 days ago

khaliqgant commented 5 days ago

Describe your changes

In an auth scenario we actually weren't deduping notifications but rather incrementing the count which doesn't make sense for an auth connection failure. Adjust logic for auth if it already has an open notification then we bail

Issue ticket number and link

NAN-1164

Checklist before requesting a review (skip if just adding/editing APIs & templates)

linear[bot] commented 5 days ago

NAN-1164 Credal Box connection failing to refresh intermittently