Closed nick-funk closed 7 months ago
None
No
INDEXES.md
The following was added to INDEXES.md:
db.notifications.createIndex({ createdAt: 1 }, { partialFilterExpression: { isDSA: { $eq: null } }, expireAfterSeconds: 30 * 24 * 60 * 60 });
Add the above index but set the expireAfterSeconds to 30 (or similar)
expireAfterSeconds
30
Post some comments and reply to them with other commenters
See your notifications show up
When the expiry time has passed, check again, they should now be gone
Report some comments as illegal
Process the reports by rejecting or completing the DSA reports
See the notifications appear in the notification area for the respective users
Wait the TTL for deletion
See the DSA notifications are never deleted
Merge into epic branch. Deploy the above mentioned index when deploying the notifications release to clients.
What does this PR do?
These changes will impact:
What changes to the GraphQL/Database Schema does this PR introduce?
None
Does this PR introduce any new environment variables or feature flags?
No
If any indexes were added, were they added to
INDEXES.md
?The following was added to
INDEXES.md
:How do I test this PR?
Add the above index but set the
expireAfterSeconds
to30
(or similar)Post some comments and reply to them with other commenters
See your notifications show up
When the expiry time has passed, check again, they should now be gone
Report some comments as illegal
Process the reports by rejecting or completing the DSA reports
See the notifications appear in the notification area for the respective users
Wait the TTL for deletion
See the DSA notifications are never deleted
Were any tests migrated to React Testing Library?
No
How do we deploy this PR?
Merge into epic branch. Deploy the above mentioned index when deploying the notifications release to clients.