Skip to content

Deployment Records Deletion#75

Draft
jfromme wants to merge 4 commits intomainfrom
fix-delete-deployment-items
Draft

Deployment Records Deletion#75
jfromme wants to merge 4 commits intomainfrom
fix-delete-deployment-items

Conversation

@jfromme
Copy link
Copy Markdown
Contributor

@jfromme jfromme commented Feb 20, 2025

Currently when an Application is deleted we delete the associated infrastructure as well as the application record in the Applications DynamoDB table. But we leave the application's deployment records in the Deployments DynamoDB table.

This PR fixes this by also deleting all deployment records associated with the deleted application.

It also uses Pusher to send a new event type, application_deletion_event, on the application's channel to notify the frontend of the application's deletion. This event looks like:

{
  "application_id" : string, the application uuid
  "time": string, the time of the deletion
  "source": string, source of event. Maybe useful in debugging?
}

…ication record, delete all linked Deployment records, and send delete event to Pusher.
@jfromme jfromme marked this pull request as draft February 20, 2025 18:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant