DEV: Promote uppy backup uploader to primary uploader (#15363)

This commit removes the enable_experimental_backup_uploader site
setting and the flags in backups-index.hbs to make the uppy
backup uploader the main one from now on.

A follow-up commit will delete the old backup uploader code and
also remove resumable.js from the project.
This commit is contained in:
Martin Brennan 2021-12-20 13:39:35 +10:00 committed by GitHub
parent b6c3e9aa03
commit d5e380e5c1
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 16 additions and 20 deletions

View File

@ -1,22 +1,8 @@
<div class="backup-options">
{{#if localBackupStorage}}
{{#if siteSettings.enable_experimental_backup_uploader}}
{{uppy-backup-uploader done=(route-action "uploadSuccess") localBackupStorage=localBackupStorage}}
{{else}}
{{resumable-upload
target="/admin/backups/upload"
success=(route-action "uploadSuccess")
error=(route-action "uploadError")
uploadText=uploadLabel
title="admin.backups.upload.title"
class="btn-default"}}
{{/if}}
{{uppy-backup-uploader done=(route-action "uploadSuccess") localBackupStorage=localBackupStorage}}
{{else}}
{{#if siteSettings.enable_experimental_backup_uploader}}
{{uppy-backup-uploader done=(route-action "remoteUploadSuccess")}}
{{else}}
{{backup-uploader done=(route-action "remoteUploadSuccess")}}
{{/if}}
{{uppy-backup-uploader done=(route-action "remoteUploadSuccess")}}
{{/if}}
{{#if site.isReadOnly}}

View File

@ -273,10 +273,6 @@ basic:
client: true
default: false
hidden: true
enable_experimental_backup_uploader:
client: true
default: false
hidden: true
enable_direct_s3_uploads:
client: true
default: false

View File

@ -0,0 +1,14 @@
# frozen_string_literal: true
class RemoveExperimentalBackupUploaderSetting < ActiveRecord::Migration[6.1]
def up
execute <<~SQL
DELETE FROM site_settings
WHERE name = 'enable_experimental_backup_uploader'
SQL
end
def down
raise ActiveRecord::IrreversibleMigration
end
end