-
Notifications
You must be signed in to change notification settings - Fork 3.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: populate susbcriber in notification feed #2633
feat: populate susbcriber in notification feed #2633
Conversation
👇 Click on the image for a new way to code review
Legend |
@mocherfaoui |
@jainpawan21
|
Yes |
@jainpawan21 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hey, thank you for the contribution! 🚀
I'll give the approval if you will include the e2e test for the new subscriber field.
What change does this PR introduce?
populate the subscriber in notification center
Why was this change needed?
closes #2404