N
Novu13mo ago
Zac Clifton

Activity Feed -> Execution details does not show me which provider the notification was sent through

This prevents me from know which provider was choicen when the notification was sent, where my notifications are primarily going through, and if there was an issue with another provider that was retried to another.
No description
16 Replies
Pawan Jain
Pawan Jain13mo ago
Click on the show detail on integration instance selected You will be able to check the integration (provider) details
Zac Clifton
Zac CliftonOP13mo ago
So the part that is confusing is that we are not consistant about the naming here. Currently it is.
No description
Zac Clifton
Zac CliftonOP13mo ago
But I think should be like this
No description
Pawan Jain
Pawan Jain13mo ago
I could not find Provider selected in our code Not sure from where it is coming
Zac Clifton
Zac CliftonOP13mo ago
Thats because I edited the html in the browser to show what I think would be more clear, its a simple text change. This is not in the code base currently.
Pawan Jain
Pawan Jain13mo ago
@Engineering I see we have instance selected in code https://github.com/novuhq/novu/blob/next/apps/worker/src/app/workflow/usecases/send-message/send-message.base.ts#L108 Do we have somewhere provider one as well?
Zac Clifton
Zac CliftonOP13mo ago
What is the difference between the Integration instance and a provider? All I was saying was to change the text
Pawan Jain
Pawan Jain13mo ago
Novu has sendgrid provider but my account can have multiple sendgrid instance in one environment
Zac Clifton
Zac CliftonOP13mo ago
So then in my account I have the novu provider and a mulitple sendgrid providers which is equivalent to multiple sendgrid instances, correct?
Pawan Jain
Pawan Jain13mo ago
Yes
Zac Clifton
Zac CliftonOP13mo ago
Ok so then would you say the wording is not consistant with the rest of the application?
Pawan Jain
Pawan Jain13mo ago
Yes I feel
Prosper
Prosper13mo ago
I totally understand this We use it interchangeably these days
Zac Clifton
Zac CliftonOP13mo ago
Ok so then I will just push a quick fix to change the wording and that should resolve this
Prosper
Prosper13mo ago
Thanks Gali
Want results from more Discord servers?
Add your server