Skip to content

Conversation

@iampearceman
Copy link
Member

What changed? Why was the change needed?

I've updated the Inbox's subscriber_id to support multi-tenancy (include org_id in the subscriber_id)

@linear
Copy link

linear bot commented May 12, 2025

@netlify
Copy link

netlify bot commented May 12, 2025

Deploy Preview for dashboard-v2-novu-staging ready!

Name Link
🔨 Latest commit 0727db2
🔍 Latest deploy log https://app.netlify.com/projects/dashboard-v2-novu-staging/deploys/682b35347d5da700080ff22d
😎 Deploy Preview https://deploy-preview-8278.dashboard-v2.novu-staging.co
📱 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 project configuration.

@iampearceman iampearceman changed the title feat: updated the subscriberid to support multi tenancy (MRK-700) feat(dashboard): updated the subscriberid to support multi tenancy (MRK-700) May 12, 2025
Co-authored-by: Dima Grossman <dima@grossman.io>
@scopsy
Copy link
Contributor

scopsy commented May 12, 2025

@iampearceman looks good! I've added another suggestion that should fix the failing CI

@scopsy
Copy link
Contributor

scopsy commented May 14, 2025

@iampearceman ping

Co-authored-by: Dima Grossman <dima@grossman.io>
@iampearceman iampearceman self-assigned this May 19, 2025
@scopsy scopsy merged commit c47dfcc into next May 19, 2025
26 checks passed
@scopsy scopsy deleted the MRK-700/emil branch May 19, 2025 14:42
bricehemery pushed a commit to jack-agency/novu that referenced this pull request Jul 7, 2025
…RK-700) (novuhq#8278)

Co-authored-by: Dima Grossman <dima@grossman.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants