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

Implement occasional maintenance task to remove old non-critical data (request and response events) #87

Open
rolznz opened this issue May 31, 2024 · 1 comment · May be fixed by getAlby/nostr-wallet-connect-next#388
Open

Implement occasional maintenance task to remove old non-critical data (request and response events) #87

rolznz opened this issue May 31, 2024 · 1 comment · May be fixed by getAlby/nostr-wallet-connect-next#388
Assignees
@rdmitr

Comments

@rolznz
Copy link
Contributor

rolznz commented May 31, 2024

We have DB constraints (currently 1GB) and the NWC database will grow over time, especially due to storing NIP-47 requests and responses. Ones older than 3 months could be occasionally deleted.
@rdmitr rdmitr linked a pull request Jun 5, 2024 that will close this issue
Cleanup stale events in the DB (#361) getAlby/nostr-wallet-connect-next#388
Draft
@rolznz
Copy link
Contributor Author

rolznz commented Jun 11, 2024 •
edited
Loading

Because this requires a migration (and might be easier to just drop and create a new table since our app is still unpublished) we would also like to change the table to transactions and include incoming payments as well (as per #105) and make sure we have enough info to do better budgets (#122) plus this requires more planning to make sure we have everything transaction-related we need for Alby Hub.

For now we are postponing this (but is still important)

@rolznz rolznz linked a pull request Jun 11, 2024 that will close this issue
Draft
@im-adithya im-adithya transferred this issue from getAlby/nostr-wallet-connect-next Jul 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rdmitr rdmitr

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Cleanup stale events in the DB (#361) getAlby/nostr-wallet-connect-next
2 participants
@rdmitr @rolznz