novuhq / novu

Open-Source Notification Platform. Embeddable Notification Center, E-mail, Push and Slack Integrations.
https://novu.co
Other
34.47k stars 3.51k forks source link

[NV-2568] πŸš€ Feature: `markAs` by `transactionId` #3763

Open vensauro opened 1 year ago

vensauro commented 1 year ago

πŸ”– Feature description

Have a way of marking a subscriber feed message as Seen with some reference outside of the novu data.

🎀 Why is this feature needed ?

Having a way of marking the feed message as seen without the messageId and with a chosen data can make easier and intuitive to use novu with a application. If the subscriber of some notification have seen the data/page that triggered the notification before opening the notification, the application can mark as seen using the internal data id (using the transactionId Like in a chat/comments where the subscriber can open the chat/comments channel before seeing the notifications panel, and now we need to fetch all notifications, filter by transactionId and mark as seen or read.

✌️ How do you aim to achieve this?

having another bodyId or endpoint on the novu API for marking a feed message as seen/read by a custom field

πŸ”„οΈ Additional Information

I have fetched on the client all feed messages and marking as read manualy the ones i know that have been readed when opening a page

πŸ‘€ Have you spent some time to check if this feature request has been raised before?

🏒 Have you read the Code of Conduct?

Are you willing to submit PR?

None

NV-2568

vensauro commented 1 year ago

i can made a PR if this is a valid request and no ones want to made this before i have some time. But for now i cannot try this, because i know a little how the novu code works, but donΒ΄t have confidence to make something