Make failed api imports retryable

app/controllers/imports/retry_controller.rb

@@ -0,0 +1,10 @@
+class Imports::RetryController < ApplicationController
+  def create
+    @import = current_user.imports.find(params[:import_id])
+    if @import.retriable? && @import.retry
+      redirect_to @import, notice: "Import being retried."
+    else
+      redirect_to @import, alert: "Import failed to retry."
+    end
+  end
+end

app/helpers/imports_helper.rb

@@ -1,4 +1,26 @@
 module ImportsHelper
+  def secondary_import_actions(import)
+    actions = []
+    if import.retriable?
+      actions << {
+        content: t("actions.retry", resource: resource_name_for(Import)),
+        url: import_retry_path(import),
+        data: {
+          turbo_method: "post"
+        }
+      }
+    end
+    actions << {
+      content: t("actions.delete", resource: resource_name_for(Import)),
+      destructive: true,
+      data: {
+        controller: "polaris",
+        target: "#destroy-modal",
+        action: "polaris#openModal"
+      }
+    }
+  end
+
   def metrics_date_range_text(import)
     if import.metrics.empty?
       t("imports.no_metrics")

app/models/import.rb

@@ -36,6 +36,15 @@ class Import < ApplicationRecord
 
   scope :in_progress, -> { where(status: %i[scheduled importing calculating]) }
 
+  def retriable?
+    shopify_payments_api_source? && failed? && user.imports.in_progress.empty?
+  end
+
+  def retry
+    update(status: :draft)
+    schedule
+  end
+
   def schedule
     return unless draft?
     scheduled!

app/views/imports/show.html.erb

@@ -6,17 +6,7 @@
   subtitle: @import.created_at.to_fs(:long),
   id: dom_id(@import),
   back_url: imports_path,
-  secondary_actions: [
-    {
-      content: t("actions.delete", resource: resource_name_for(Import)),
-      destructive: true,
-      data: {
-        controller: "polaris",
-        target: "#destroy-modal",
-        action: "polaris#openModal"
-      }
-    }
-  ],
+  secondary_actions: secondary_import_actions(@import),
 ) do |page| %>
 
   <% page.with_title_metadata do %>

config/locales/en.yml

@@ -10,6 +10,7 @@ en:
     delete_all: "Delete all %{resource}"
     more_actions: "More actions"
     add: "Add %{resource}"
+    retry: "Retry %{resource}"
   statuses:
     draft: "Draft"
     scheduled: "Scheduled"

config/routes.rb

@@ -7,6 +7,7 @@
 
   resources :imports, only: [:index, :show, :new, :create, :destroy] do
     resource :globe, only: [:show], controller: "imports/globes"
+    resource :retry, only: [:create], controller: "imports/retry"
     collection do
       delete :destroy_all, to: "imports/destroy_all#destroy"
     end

test/controllers/imports/retry_controller_test.rb

@@ -0,0 +1,16 @@
+require "test_helper"
+
+class Imports::RetryControllerTest < ActionDispatch::IntegrationTest
+  setup do
+    sign_in users(:regular)
+    @import = imports(:failed_shopify_payments_api)
+  end
+
+  test "should retry shopify api imports" do
+    post import_retry_url(@import)
+
+    assert @import.reload.draft? || @import.reload.scheduled?
+    assert_redirected_to import_url(@import)
+    assert flash[:notice], "Import being retried."
+  end
+end

test/fixtures/imports.yml

@@ -9,3 +9,8 @@ failed:
   source: <%= Import.sources[:csv_file] %>
   status: failed
   user: regular
+
+failed_shopify_payments_api:
+  source: <%= Import.sources[:shopify_payments_api] %>
+  status: failed
+  user: regular