fastlane 2.188.0 → 2.192.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 23a8e6b064fc103d7a8d701428ac360a7cb03ef0761d6dc067ae392dacd02bbf
-  data.tar.gz: 11a4781c02ca5777fceac1364311be8726f6a4fc6f8943f5215a9c753c66c127
+  metadata.gz: f1c0c051ec3343adfcc8dcc927954b9fe5e76bf56d813be5d4f0823695dcb835
+  data.tar.gz: 328239727a6f80cca28a6524784cb2f4e1b39454f0c6f2e2cf036735c26f6811
 SHA512:
-  metadata.gz: f4ebfae993bfb3db9bc6dcf91c4d526b98b9202c6813f894579979ad52a9bffe74522eaa8b2a2a36b7a0d87be275cac4ce677a9d9864746bc92d5c7d64737e12
-  data.tar.gz: b1aacae9f83f897fd56bc9792d3a7c9842f0ffcf7caf64963b80dd7283f304116232d46608512af53804fdbb6075b1dffab7339776c0a1cff85b75992f5bf53e
+  metadata.gz: c430d54e72ceb927d096d02ac12578e196194405a6398768656ded1b0274d6fdb1988487d400b6c3e7434e02b82f372ac200959f524afaa9374d385a4ab00a46
+  data.tar.gz: 3800cfae6b4add96d08f98fa780c721d2444c4e81e1bf24e437e5e7e9f455682b198a7bc63246738fd758da7f0437dfb311d410632b0d22faa4af0e58e7d0301

data/README.md

@@ -35,49 +35,29 @@ If the above doesn't help, please [submit an issue](https://github.com/fastlane/
 <!-- This table is regenerated and resorted on each release -->
 <table id='team'>
 <tr>
-<td id='matthew-ellis'>
-<a href='https://github.com/matthewellis'>
-<img src='https://github.com/matthewellis.png' width='140px;'>
-</a>
-<h4 align='center'><a href='https://twitter.com/mellis1995'>Matthew Ellis</a></h4>
-</td>
-<td id='josh-holtz'>
-<a href='https://github.com/joshdholtz'>
-<img src='https://github.com/joshdholtz.png' width='140px;'>
-</a>
-<h4 align='center'><a href='https://twitter.com/joshdholtz'>Josh Holtz</a></h4>
-</td>
-<td id='maksym-grebenets'>
-<a href='https://github.com/mgrebenets'>
-<img src='https://github.com/mgrebenets.png' width='140px;'>
-</a>
-<h4 align='center'><a href='https://twitter.com/mgrebenets'>Maksym Grebenets</a></h4>
-</td>
 <td id='stefan-natchev'>
 <a href='https://github.com/snatchev'>
 <img src='https://github.com/snatchev.png' width='140px;'>
 </a>
 <h4 align='center'><a href='https://twitter.com/snatchev'>Stefan Natchev</a></h4>
 </td>
-<td id='roger-oba'>
-<a href='https://github.com/rogerluan'>
-<img src='https://github.com/rogerluan.png' width='140px;'>
+<td id='maksym-grebenets'>
+<a href='https://github.com/mgrebenets'>
+<img src='https://github.com/mgrebenets.png' width='140px;'>
 </a>
-<h4 align='center'><a href='https://twitter.com/rogerluan_'>Roger Oba</a></h4>
+<h4 align='center'><a href='https://twitter.com/mgrebenets'>Maksym Grebenets</a></h4>
 </td>
-</tr>
-<tr>
-<td id='manish-rathi'>
-<a href='https://github.com/crazymanish'>
-<img src='https://github.com/crazymanish.png' width='140px;'>
+<td id='aaron-brager'>
+<a href='https://github.com/getaaron'>
+<img src='https://github.com/getaaron.png' width='140px;'>
 </a>
-<h4 align='center'><a href='https://twitter.com/iammanishrathi'>Manish Rathi</a></h4>
+<h4 align='center'><a href='https://twitter.com/getaaron'>Aaron Brager</a></h4>
 </td>
-<td id='danielle-tomlinson'>
-<a href='https://github.com/endocrimes'>
-<img src='https://github.com/endocrimes.png' width='140px;'>
+<td id='satoshi-namai'>
+<a href='https://github.com/ainame'>
+<img src='https://github.com/ainame.png' width='140px;'>
 </a>
-<h4 align='center'><a href='https://twitter.com/endocrimes'>Danielle Tomlinson</a></h4>
+<h4 align='center'><a href='https://twitter.com/ainame'>Satoshi Namai</a></h4>
 </td>
 <td id='jorge-revuelta-h'>
 <a href='https://github.com/minuscorp'>
@@ -85,37 +65,19 @@ If the above doesn't help, please [submit an issue](https://github.com/fastlane/
 </a>
 <h4 align='center'><a href='https://twitter.com/minuscorp'>Jorge Revuelta H</a></h4>
 </td>
-<td id='jimmy-dee'>
-<a href='https://github.com/jdee'>
-<img src='https://github.com/jdee.png' width='140px;'>
-</a>
-<h4 align='center'>Jimmy Dee</h4>
-</td>
-<td id='iulian-onofrei'>
-<a href='https://github.com/revolter'>
-<img src='https://github.com/revolter.png' width='140px;'>
-</a>
-<h4 align='center'><a href='https://twitter.com/Revolt666'>Iulian Onofrei</a></h4>
-</td>
 </tr>
 <tr>
-<td id='helmut-januschka'>
-<a href='https://github.com/hjanuschka'>
-<img src='https://github.com/hjanuschka.png' width='140px;'>
-</a>
-<h4 align='center'><a href='https://twitter.com/hjanuschka'>Helmut Januschka</a></h4>
-</td>
-<td id='satoshi-namai'>
-<a href='https://github.com/ainame'>
-<img src='https://github.com/ainame.png' width='140px;'>
+<td id='olivier-halligon'>
+<a href='https://github.com/AliSoftware'>
+<img src='https://github.com/AliSoftware.png' width='140px;'>
 </a>
-<h4 align='center'><a href='https://twitter.com/ainame'>Satoshi Namai</a></h4>
+<h4 align='center'><a href='https://twitter.com/aligatr'>Olivier Halligon</a></h4>
 </td>
-<td id='luka-mirosevic'>
-<a href='https://github.com/lmirosevic'>
-<img src='https://github.com/lmirosevic.png' width='140px;'>
+<td id='manu-wallner'>
+<a href='https://github.com/milch'>
+<img src='https://github.com/milch.png' width='140px;'>
 </a>
-<h4 align='center'><a href='https://twitter.com/lmirosevic'>Luka Mirosevic</a></h4>
+<h4 align='center'><a href='https://twitter.com/acrooow'>Manu Wallner</a></h4>
 </td>
 <td id='joshua-liebowitz'>
 <a href='https://github.com/taquitos'>
@@ -123,57 +85,75 @@ If the above doesn't help, please [submit an issue](https://github.com/fastlane/
 </a>
 <h4 align='center'><a href='https://twitter.com/taquitos'>Joshua Liebowitz</a></h4>
 </td>
-<td id='andrew-mcburney'>
-<a href='https://github.com/armcburney'>
-<img src='https://github.com/armcburney.png' width='140px;'>
+<td id='luka-mirosevic'>
+<a href='https://github.com/lmirosevic'>
+<img src='https://github.com/lmirosevic.png' width='140px;'>
 </a>
-<h4 align='center'><a href='https://twitter.com/armcburney'>Andrew McBurney</a></h4>
+<h4 align='center'><a href='https://twitter.com/lmirosevic'>Luka Mirosevic</a></h4>
 </td>
-</tr>
-<tr>
-<td id='olivier-halligon'>
-<a href='https://github.com/AliSoftware'>
-<img src='https://github.com/AliSoftware.png' width='140px;'>
+<td id='jan-piotrowski'>
+<a href='https://github.com/janpio'>
+<img src='https://github.com/janpio.png' width='140px;'>
 </a>
-<h4 align='center'><a href='https://twitter.com/aligatr'>Olivier Halligon</a></h4>
+<h4 align='center'><a href='https://twitter.com/Sujan'>Jan Piotrowski</a></h4>
 </td>
+</tr>
+<tr>
 <td id='daniel-jankowski'>
 <a href='https://github.com/mollyIV'>
 <img src='https://github.com/mollyIV.png' width='140px;'>
 </a>
 <h4 align='center'><a href='https://twitter.com/mollyIV'>Daniel Jankowski</a></h4>
 </td>
+<td id='helmut-januschka'>
+<a href='https://github.com/hjanuschka'>
+<img src='https://github.com/hjanuschka.png' width='140px;'>
+</a>
+<h4 align='center'><a href='https://twitter.com/hjanuschka'>Helmut Januschka</a></h4>
+</td>
 <td id='jérôme-lacoste'>
 <a href='https://github.com/lacostej'>
 <img src='https://github.com/lacostej.png' width='140px;'>
 </a>
 <h4 align='center'><a href='https://twitter.com/lacostej'>Jérôme Lacoste</a></h4>
 </td>
-<td id='jan-piotrowski'>
-<a href='https://github.com/janpio'>
-<img src='https://github.com/janpio.png' width='140px;'>
+<td id='matthew-ellis'>
+<a href='https://github.com/matthewellis'>
+<img src='https://github.com/matthewellis.png' width='140px;'>
 </a>
-<h4 align='center'><a href='https://twitter.com/Sujan'>Jan Piotrowski</a></h4>
+<h4 align='center'><a href='https://twitter.com/mellis1995'>Matthew Ellis</a></h4>
 </td>
-<td id='manu-wallner'>
-<a href='https://github.com/milch'>
-<img src='https://github.com/milch.png' width='140px;'>
+<td id='fumiya-nakamura'>
+<a href='https://github.com/nafu'>
+<img src='https://github.com/nafu.png' width='140px;'>
 </a>
-<h4 align='center'><a href='https://twitter.com/acrooow'>Manu Wallner</a></h4>
+<h4 align='center'><a href='https://twitter.com/nafu003'>Fumiya Nakamura</a></h4>
 </td>
 </tr>
 <tr>
-<td id='max-ott'>
-<a href='https://github.com/max-ott'>
-<img src='https://github.com/max-ott.png' width='140px;'>
+<td id='danielle-tomlinson'>
+<a href='https://github.com/endocrimes'>
+<img src='https://github.com/endocrimes.png' width='140px;'>
 </a>
-<h4 align='center'><a href='https://twitter.com/ott_max'>Max Ott</a></h4>
+<h4 align='center'><a href='https://twitter.com/endocrimes'>Danielle Tomlinson</a></h4>
 </td>
-<td id='aaron-brager'>
-<a href='https://github.com/getaaron'>
-<img src='https://github.com/getaaron.png' width='140px;'>
+<td id='roger-oba'>
+<a href='https://github.com/rogerluan'>
+<img src='https://github.com/rogerluan.png' width='140px;'>
 </a>
-<h4 align='center'><a href='https://twitter.com/getaaron'>Aaron Brager</a></h4>
+<h4 align='center'><a href='https://twitter.com/rogerluan_'>Roger Oba</a></h4>
+</td>
+<td id='iulian-onofrei'>
+<a href='https://github.com/revolter'>
+<img src='https://github.com/revolter.png' width='140px;'>
+</a>
+<h4 align='center'><a href='https://twitter.com/Revolt666'>Iulian Onofrei</a></h4>
+</td>
+<td id='jimmy-dee'>
+<a href='https://github.com/jdee'>
+<img src='https://github.com/jdee.png' width='140px;'>
+</a>
+<h4 align='center'>Jimmy Dee</h4>
 </td>
 <td id='felix-krause'>
 <a href='https://github.com/KrauseFx'>
@@ -181,17 +161,37 @@ If the above doesn't help, please [submit an issue](https://github.com/fastlane/
 </a>
 <h4 align='center'><a href='https://twitter.com/KrauseFx'>Felix Krause</a></h4>
 </td>
+</tr>
+<tr>
+<td id='max-ott'>
+<a href='https://github.com/max-ott'>
+<img src='https://github.com/max-ott.png' width='140px;'>
+</a>
+<h4 align='center'><a href='https://twitter.com/ott_max'>Max Ott</a></h4>
+</td>
+<td id='manish-rathi'>
+<a href='https://github.com/crazymanish'>
+<img src='https://github.com/crazymanish.png' width='140px;'>
+</a>
+<h4 align='center'><a href='https://twitter.com/iammanishrathi'>Manish Rathi</a></h4>
+</td>
 <td id='kohki-miki'>
 <a href='https://github.com/giginet'>
 <img src='https://github.com/giginet.png' width='140px;'>
 </a>
 <h4 align='center'><a href='https://twitter.com/giginet'>Kohki Miki</a></h4>
 </td>
-<td id='fumiya-nakamura'>
-<a href='https://github.com/nafu'>
-<img src='https://github.com/nafu.png' width='140px;'>
+<td id='andrew-mcburney'>
+<a href='https://github.com/armcburney'>
+<img src='https://github.com/armcburney.png' width='140px;'>
 </a>
-<h4 align='center'><a href='https://twitter.com/nafu003'>Fumiya Nakamura</a></h4>
+<h4 align='center'><a href='https://twitter.com/armcburney'>Andrew McBurney</a></h4>
+</td>
+<td id='josh-holtz'>
+<a href='https://github.com/joshdholtz'>
+<img src='https://github.com/joshdholtz.png' width='140px;'>
+</a>
+<h4 align='center'><a href='https://twitter.com/joshdholtz'>Josh Holtz</a></h4>
 </td>
 </tr>
 </table>

data/deliver/lib/deliver/app_screenshot.rb

@@ -324,7 +324,8 @@ module Deliver
       is_3rd_gen = [
         "iPad Pro (12.9-inch) (3rd generation)", # default simulator name has this
         "iPad Pro (12.9-inch) (4th generation)", # default simulator name has this
-        "ipadPro129" # downloaded screenshots name has this
+        "ipadPro129", # downloaded screenshots name has this,
+        "3GEN" # downloaded screenshots name from App Store Connect API has this
       ].any? { |key| filename.include?(key) }
       if is_3rd_gen
         if screen_size == ScreenSize::IOS_IPAD_PRO

data/deliver/lib/deliver/app_screenshot_iterator.rb

@@ -85,8 +85,8 @@ module Deliver
           app_screenshot_set ||= localization.create_app_screenshot_set(attributes: { screenshotDisplayType: display_type })
 
           # iterate over screenshots per display size with index
-          screenshots.each do |screenshot|
-            yield(localization, app_screenshot_set, screenshot)
+          screenshots.each.with_index do |screenshot, index|
+            yield(localization, app_screenshot_set, screenshot, index)
           end
         end
       end

data/deliver/lib/deliver/loader.rb

@@ -82,7 +82,7 @@ module Deliver
     #
     # @param root [String] A directory path
     # @param ignore_validation [String] Set false not to raise the error when finding invalid folder name
-    # @return [Array<AppScreenshot>] The list of AppScreenshot that exist under given `root` directory
+    # @return [Array<Deliver::AppScreenshot>] The list of AppScreenshot that exist under given `root` directory
     def self.load_app_screenshots(root, ignore_validation)
       screenshots = language_folders(root, ignore_validation, true).flat_map do |language_folder|
         paths = if language_folder.framed_file_paths.count > 0

data/deliver/lib/deliver/options.rb

@@ -162,6 +162,12 @@ module Deliver
                                      description: "Clear all previously uploaded screenshots before uploading the new ones",
                                      type: Boolean,
                                      default_value: false),
+        FastlaneCore::ConfigItem.new(key: :sync_screenshots,
+                                     env_name: "DELIVER_SYNC_SCREENSHOTS",
+                                     description: "Sync screenshots with local ones. This is currently beta option" \
+                                                  "so set true to 'FASTLANE_ENABLE_BETA_DELIVER_SYNC_SCREENSHOTS' environment variable as well",
+                                     type: Boolean,
+                                     default_value: false),
         FastlaneCore::ConfigItem.new(key: :submit_for_review,
                                      env_name: "DELIVER_SUBMIT_FOR_REVIEW",
                                      description: "Submit the new version for Review after uploading everything",

data/deliver/lib/deliver/runner.rb

@@ -10,6 +10,7 @@ require_relative 'submit_for_review'
 require_relative 'upload_price_tier'
 require_relative 'upload_metadata'
 require_relative 'upload_screenshots'
+require_relative 'sync_screenshots'
 require_relative 'detect_values'
 
 module Deliver
@@ -143,7 +144,14 @@ module Deliver
 
       # Commit
       upload_metadata.upload(options)
-      upload_screenshots.upload(options, screenshots)
+
+      if options[:sync_screenshots]
+        sync_screenshots = SyncScreenshots.new(app: Deliver.cache[:app], platform: Spaceship::ConnectAPI::Platform.map(options[:platform]))
+        sync_screenshots.sync(screenshots)
+      else
+        upload_screenshots.upload(options, screenshots)
+      end
+
       UploadPriceTier.new.upload(options)
     end
 

data/deliver/lib/deliver/screenshot_comparable.rb

@@ -0,0 +1,62 @@
+require 'spaceship/connect_api/models/app_screenshot'
+require 'spaceship/connect_api/models/app_screenshot_set'
+
+require_relative 'app_screenshot'
+
+module Deliver
+  # This clsas enables you to compare equality between different representations of the screenshots
+  # in the standard API `Array#-` that requires objects to implements `eql?` and `hash`.
+  class ScreenshotComparable
+    # A unique key value that is consist of locale, filename, and checksum.
+    attr_reader :key
+
+    # A hash object that contains the source data of this representation class
+    attr_reader :context
+
+    def self.create_from_local(screenshot:, app_screenshot_set:)
+      raise ArgumentError unless screenshot.kind_of?(Deliver::AppScreenshot)
+      raise ArgumentError unless app_screenshot_set.kind_of?(Spaceship::ConnectAPI::AppScreenshotSet)
+
+      new(
+        path: "#{screenshot.language}/#{File.basename(screenshot.path)}",
+        checksum: calculate_checksum(screenshot.path),
+        context: {
+          screenshot: screenshot,
+          app_screenshot_set: app_screenshot_set
+        }
+      )
+    end
+
+    def self.create_from_remote(app_screenshot:, locale:)
+      raise ArgumentError unless app_screenshot.kind_of?(Spaceship::ConnectAPI::AppScreenshot)
+      raise ArgumentError unless locale.kind_of?(String)
+
+      new(
+        path: "#{locale}/#{app_screenshot.file_name}",
+        checksum: app_screenshot.source_file_checksum,
+        context: {
+          app_screenshot: app_screenshot,
+          locale: locale
+        }
+      )
+    end
+
+    def self.calculate_checksum(path)
+      bytes = File.binread(path)
+      Digest::MD5.hexdigest(bytes)
+    end
+
+    def initialize(path:, checksum:, context:)
+      @key = "#{path}/#{checksum}"
+      @context = context
+    end
+
+    def eql?(other)
+      key == other.key
+    end
+
+    def hash
+      key.hash
+    end
+  end
+end

data/deliver/lib/deliver/sync_screenshots.rb

@@ -0,0 +1,200 @@
+require 'fastlane_core'
+require 'digest/md5'
+require 'naturally'
+
+require_relative 'app_screenshot'
+require_relative 'app_screenshot_iterator'
+require_relative 'loader'
+require_relative 'screenshot_comparable'
+
+module Deliver
+  class SyncScreenshots
+    DeleteScreenshotJob = Struct.new(:app_screenshot, :locale)
+    UploadScreenshotJob = Struct.new(:app_screenshot_set, :path)
+
+    class UploadResult
+      attr_reader :asset_delivery_state_counts, :failing_screenshots
+
+      def initialize(asset_delivery_state_counts:, failing_screenshots:)
+        @asset_delivery_state_counts = asset_delivery_state_counts
+        @failing_screenshots = failing_screenshots
+      end
+
+      def processing?
+        @asset_delivery_state_counts.fetch('UPLOAD_COMPLETE', 0) > 0
+      end
+
+      def screenshot_count
+        @asset_delivery_state_counts.fetch('COMPLETE', 0)
+      end
+    end
+
+    def initialize(app:, platform:)
+      @app = app
+      @platform = platform
+    end
+
+    def sync_from_path(screenshots_path)
+      # load local screenshots
+      screenshots = Deliver::Loader.load_app_screenshots(screenshots_path, true)
+      sync(screenshots)
+    end
+
+    def sync(screenshots)
+      UI.important('This is currently a beta feature in fastlane. This may cause some errors on your environment.')
+
+      unless FastlaneCore::Feature.enabled?('FASTLANE_ENABLE_BETA_DELIVER_SYNC_SCREENSHOTS')
+        UI.user_error!('Please set a value to "FASTLANE_ENABLE_BETA_DELIVER_SYNC_SCREENSHOTS" environment variable ' \
+                       'if you acknowleage the risk and try this out.')
+      end
+
+      UI.important("Will begin uploading snapshots for '#{version.version_string}' on App Store Connect")
+
+      # enable localizations that will be used
+      screenshots_per_language = screenshots.group_by(&:language)
+      enable_localizations(screenshots_per_language.keys)
+
+      # create iterator
+      localizations = fetch_localizations
+      iterator = Deliver::AppScreenshotIterator.new(localizations)
+
+      # sync local screenshots with remote settings by deleting and uploading
+      UI.message("Starting with the upload of screenshots...")
+      replace_screenshots(iterator, screenshots)
+
+      # ensure screenshots within screenshot sets are sorted in right order
+      sort_screenshots(iterator)
+
+      UI.important('Screenshots are synced successfully!')
+    end
+
+    def enable_localizations(locales)
+      localizations = fetch_localizations
+      locales_to_enable = locales - localizations.map(&:locale)
+      Helper.show_loading_indicator("Activating localizations for #{locales_to_enable.join(', ')}...")
+      locales_to_enable.each do |locale|
+        version.create_app_store_version_localization(attributes: { locale: locale })
+      end
+      Helper.hide_loading_indicator
+    end
+
+    def replace_screenshots(iterator, screenshots, retries = 3)
+      # delete and upload screenshots to get App Store Connect in sync
+      do_replace_screenshots(iterator, screenshots, create_delete_worker, create_upload_worker)
+
+      # wait for screenshots to be processed on App Store Connect end and
+      # ensure the number of uploaded screenshots matches the one in local
+      result = wait_for_complete(iterator)
+      return if !result.processing? && result.screenshot_count == screenshots.count
+
+      if retries.zero?
+        UI.crash!("Retried uploading screenshots #{retries} but there are still failures of processing screenshots." \
+                  "Check App Store Connect console to work out which screenshots processed unsuccessfully.")
+      end
+
+      # retry with deleting failing screenshots
+      result.failing_screenshots.each(&:delete!)
+      replace_screenshots(iterator, screenshots, retries - 1)
+    end
+
+    # This is a testable method that focuses on figuring out what to update
+    def do_replace_screenshots(iterator, screenshots, delete_worker, upload_worker)
+      remote_screenshots = iterator.each_app_screenshot.map do |localization, app_screenshot_set, app_screenshot|
+        ScreenshotComparable.create_from_remote(app_screenshot: app_screenshot, locale: localization.locale)
+      end
+
+      local_screenshots = iterator.each_local_screenshot(screenshots.group_by(&:language)).map do |localization, app_screenshot_set, screenshot, index|
+        if index >= 10
+          UI.user_error!("Found #{localization.locale} has more than 10 screenshots for #{app_screenshot_set.screenshot_display_type}. "\
+                         "Make sure containts only necessary screenshots.")
+        end
+        ScreenshotComparable.create_from_local(screenshot: screenshot, app_screenshot_set: app_screenshot_set)
+      end
+
+      # Thanks to `Array#-` API and `ScreenshotComparable`, working out diffs between local screenshot directory and App Store Connect
+      # is as easy as you can see below. The former one finds what is missing in local and the latter one is visa versa.
+      screenshots_to_delete = remote_screenshots - local_screenshots
+      screenshots_to_upload = local_screenshots - remote_screenshots
+
+      delete_jobs = screenshots_to_delete.map { |x| DeleteScreenshotJob.new(x.context[:app_screenshot], x.context[:locale]) }
+      delete_worker.batch_enqueue(delete_jobs)
+      delete_worker.start
+
+      upload_jobs = screenshots_to_upload.map { |x| UploadScreenshotJob.new(x.context[:app_screenshot_set], x.context[:screenshot].path) }
+      upload_worker.batch_enqueue(upload_jobs)
+      upload_worker.start
+    end
+
+    def wait_for_complete(iterator)
+      retry_count = 0
+      Helper.show_loading_indicator("Waiting for all the screenshots processed...")
+      loop do
+        failing_screenshots = []
+        state_counts = iterator.each_app_screenshot.map { |_, _, app_screenshot| app_screenshot }.each_with_object({}) do |app_screenshot, hash|
+          state = app_screenshot.asset_delivery_state['state']
+          hash[state] ||= 0
+          hash[state] += 1
+          failing_screenshots << app_screenshot if app_screenshot.error?
+        end
+
+        result = UploadResult.new(asset_delivery_state_counts: state_counts, failing_screenshots: failing_screenshots)
+        return result unless result.processing?
+
+        # sleep with exponential backoff
+        interval = 5 + (2**retry_count)
+        UI.message("There are still incomplete screenshots. Will check the states again in #{interval} secs - #{state_counts}")
+        sleep(interval)
+        retry_count += 1
+      end
+    ensure
+      Helper.hide_loading_indicator
+    end
+
+    def sort_screenshots(iterator)
+      Helper.show_loading_indicator("Sorting screenshots uploaded...")
+      sort_worker = create_sort_worker
+      sort_worker.batch_enqueue(iterator.each_app_screenshot_set.to_a.map { |_, set| set })
+      sort_worker.start
+      Helper.hide_loading_indicator
+    end
+
+    private
+
+    def version
+      @version ||= @app.get_edit_app_store_version(platform: @platform)
+    end
+
+    def fetch_localizations
+      version.get_app_store_version_localizations
+    end
+
+    def create_upload_worker
+      FastlaneCore::QueueWorker.new do |job|
+        UI.verbose("Uploading '#{job.path}'...")
+        start_time = Time.now
+        job.app_screenshot_set.upload_screenshot(path: job.path, wait_for_processing: false)
+        UI.message("Uploaded '#{job.path}'... (#{Time.now - start_time} secs)")
+      end
+    end
+
+    def create_delete_worker
+      FastlaneCore::QueueWorker.new do |job|
+        target = "id=#{job.app_screenshot.id} #{job.locale} #{job.app_screenshot.file_name}"
+        UI.verbose("Deleting '#{target}'")
+        start_time = Time.now
+        job.app_screenshot.delete!
+        UI.message("Deleted '#{target}' -  (#{Time.now - start_time} secs)")
+      end
+    end
+
+    def create_sort_worker
+      FastlaneCore::QueueWorker.new do |app_screenshot_set|
+        original_ids = app_screenshot_set.app_screenshots.map(&:id)
+        sorted_ids = Naturally.sort(app_screenshot_set.app_screenshots, by: :file_name).map(&:id)
+        if original_ids != sorted_ids
+          app_screenshot_set.reorder_screenshots(app_screenshot_ids: sorted_ids)
+        end
+      end
+    end
+  end
+end