No relay deletion endpoint or UI #31

Open
opened 2026-04-20 11:41:33 +00:00 by userAdityaa · 2 comments
Contributor

There is no DELETE /relays/:id endpoint in the backend router and no delete action in any frontend component. Users who want to clean up test relays, renamed relays, or abandoned relays have no way to do so. Inactive relays accumulate in the database and in the relay list UI indefinitely.

There is no `DELETE /relays/:id` endpoint in the backend router and no delete action in any frontend component. Users who want to clean up test relays, renamed relays, or abandoned relays have no way to do so. Inactive relays accumulate in the database and in the relay list UI indefinitely.
Owner

I'm not sure about this one. I agree it would be nice to clean up clutter, but I think it's also important to implement soft delete so that we have a record of stuff that's been done in the past. I would have set up inactivate to delete the zooid config file, but we don't keep a copy of the relay secret, so that would require migrating relay secrets on reactivation (and data loss because the database would be deleted).

Let's do this: add an option to delete a relay, but make that a new relay status, instead of something that actually deletes the row. However, on delete, actually delete the zooid config file (and database? I have been thinking about it and I think we probably need to refactor zooid's db layer). Deleted relays can't be un-deleted, and only inactive relays can be deleted.

I'm not sure about this one. I agree it would be nice to clean up clutter, but I think it's also important to implement soft delete so that we have a record of stuff that's been done in the past. I would have set up inactivate to delete the zooid config file, but we don't keep a copy of the relay secret, so that would require migrating relay secrets on reactivation (and data loss because the database would be deleted). Let's do this: add an option to delete a relay, but make that a new relay status, instead of something that actually deletes the row. However, on delete, actually delete the zooid config file (and database? I have been thinking about it and I think we probably need to refactor zooid's db layer). Deleted relays can't be un-deleted, and only inactive relays can be deleted.
Author
Contributor

Marking this as low priority for now, since it’s an enhancement rather than something blocking the launch.

Marking this as low priority for now, since it’s an enhancement rather than something blocking the launch.
Sign in to join this conversation.
No Label
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: coracle/caravel#31