🐛 Bug Report: step conditions not working with push webhook provider

[jainpawan21]

📜 Description

Novu supports adding multiple integrations of same provider. Each integration can have step conditions on tenants and based on these conditions, these providers can be used in workflow execution.

👟 Reproduction steps

1. Create two push webhook integrations, give different webhook URL and secret hmac key. Give different names to each one to make it easy to identify later. Make these two integrations active
2. Create a subscriber push-webhook-subscriber for testing this bug
3. Update the above subscriber's credentials using this API for both of the integrations. Use any string value for device token
4. Create a workflow with only push step. write title and message content.
5. make sure only above two push webhook integrations are active for push channel providers
6. Create two tenants with different name and identifier. Store this data

// for first tenant
{
  "country": "US"
}

// for second tenant
{
  "country": "EU"
}

7. Add conditions on tenants for data.country key on both integrations

8. Trigger the workflow to push-webhook-subscriber subscriber and send tenant identifier also
9. Push step will be executed for both providers and conditions will not take effect

👍 Expected behavior

tenants conditions should take effect and integration should be disabled based on conditions

👎 Actual Behavior with Screenshots

Tenants conditions are not working

Novu version

Novu SaaS

npm version

No response

node version

No response

📃 Provide any additional context for the Bug.

transactionId: - b67f3197-f0b5-4ea5-9603-f82d332cc3c1

👀 Have you spent some time to check if this bug has been raised before?

• I checked and didn't find a similar issue

🏢 Have you read the Contributing Guidelines?

• I have read the Contributing Guidelines

Are you willing to submit PR?

None

[linear]

NV-3785 🐛 Bug Report: step conditions not working with push webhook provider