🐛 Bug Report: Notifications with deleted: false are not returned by queries (self-hosted)

[katyaterletskaya]

📜 Description

Query results exclude documents where the "deleted" field is set to false. This issue seems to have arisen following the commit, which introduced new pre-hook logic that specifies deleted: { $exists: false }.

I have not been able to find any migration scripts designed to remove the "deleted" field, which may help mitigate this concern. I would appreciate guidance on where to find the relevant migration scripts or recommendations for addressing this issue.

👟 Reproduction steps

1. Query notifications from the Message repository
2. Observe that documents with deleted: false are not returned

👍 Expected behavior

Queries should return all non-deleted messages, including those where deleted is explicitly set to false.

👎 Actual Behavior with Screenshots

Documents with deleted: false are excluded from query results due to the deleted: { $exists: false } condition.

Novu version

3.12.0

npm version

10.8.2

node version

20.19.0

📃 Provide any additional context for the Bug.

No response

👀 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-7087

[scopsy]

Hi Katya, the the delete plugin was delete quite a while ago from the messages repository due to the message collection performance. If those are old messages on your end, the quickest way would be to remove or $unset the deleted field from your collection (or hard deleted the message items)

[katyaterletskaya]

Hi @scopsy 😊 Thanks so much for your reply and for the helpful suggestions on resolving the issue!