Skip to content

feat(site): skip the drop backup for failed new sites and from the drop dialog - #535

Open
mihir-kandoi wants to merge 3 commits into
fix/site-drop-db-type-no-backupfrom
fix/drop-site-skip-backup-followup
Open

mihir-kandoi wants to merge 3 commits into
fix/site-drop-db-type-no-backupfrom
fix/drop-site-skip-backup-followup

Conversation

@mihir-kandoi

@mihir-kandoi mihir-kandoi commented Sep 26, 2026 •

Copy link
Copy Markdown
Contributor

Stacked on #534.

  • A failed or cancelled new site is dropped with no_backup=True; it has no user data, so the full dump was wasted work.
  • The drop-site dialog gets a "Take a backup before dropping" checkbox, checked by default. Unchecking it sends DELETE /sites/<name>?no_backup=1.
image

A new site that failed or was cancelled holds no user data, so the full
dump frappe drop-site takes first is wasted work and leaves a junk backup.
Adds a "Take a backup before dropping" checkbox, checked by default.
Unchecking it sends DELETE /sites/<name>?no_backup=1, so large sites can
be dropped without frappe taking a full dump first.
@mihir-kandoi
mihir-kandoi added this pull request to stack #536 September 26, 2026 08:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant