Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Notifications] Delete old NosNotification objects #424

Closed
mplorentz opened this issue Jul 14, 2023 · 1 comment
Closed

[Notifications] Delete old NosNotification objects #424

mplorentz opened this issue Jul 14, 2023 · 1 comment
Assignees
@martindsq
Labels
bug Something isn't working devx Work that will improve developer experience/velocity

Comments

@mplorentz
Copy link
Member

mplorentz commented Jul 14, 2023 •
edited
Loading

When I wrote the push notification code I skipped writing code to delete old NosNotification objects from Core Data once we no longer need them. This will cause database sizes to slowly expand infinitely if we don't fix it.
@mplorentz mplorentz converted this from a draft issue Jul 14, 2023
@mplorentz mplorentz added the devx Work that will improve developer experience/velocity label Jul 14, 2023
@mplorentz mplorentz moved this to Delivery Backlog in Verse: Nos Projects Jul 18, 2023
@setch-l setch-l changed the title Delete old NosNotification objects [Notifications] Delete old NosNotification objects Dec 12, 2023
@setch-l setch-l added the bug Something isn't working label Dec 12, 2023
@setch-l setch-l moved this from Delivery Backlog to Dev Sprint in Verse: Nos Projects Dec 12, 2023
@martindsq martindsq moved this from Dev Sprint to In Progress in Verse: Nos Projects Dec 18, 2023
@martindsq martindsq mentioned this issue Dec 22, 2023
Merged
@martindsq martindsq moved this from In Progress to Code Review in Verse: Nos Projects Dec 22, 2023
@martindsq martindsq moved this from Code Review to UAT in Verse: Nos Projects Jan 15, 2024
@setch-l setch-l moved this from UAT to Done in Verse: Nos Projects Jan 17, 2024
@joshuatbrown
Copy link
Contributor

Closing, as this looks like it was completed in #765.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@martindsq martindsq

Labels
bug Something isn't working devx Work that will improve developer experience/velocity
Projects
Verse: Nos Projects
Archived in project
Milestone
No milestone
Development

No branches or pull requests
4 participants
@joshuatbrown @mplorentz @martindsq @setch-l