Added new deliver download_screenshots action

bin/deliver

@@ -4,6 +4,7 @@ $:.push File.expand_path("../../lib", __FILE__)
 
 require 'deliver'
 require 'commander'
+require 'deliver/download_screenshots'
 
 HighLine.track_eof = false
 
@@ -43,18 +44,18 @@ class FastlaneApplication
 
     def run_deliver(options)
       path = (Deliver::Helper.fastlane_enabled?? './fastlane' : '.')
-        Dir.chdir(path) do # switch the context
-          if File.exists?(deliver_path)
-            # Everything looks alright, use the given Deliverfile
-            options.default :beta => false, :skip_deploy => false
-            Deliver::Deliverer.new(deliver_path, force: options.force, is_beta_ipa: options.beta, skip_deploy: options.skip_deploy)
-          else
-            Deliver::Helper.log.warn("No Deliverfile found at path '#{deliver_path}'.")
-            if agree("Do you want to create a new Deliverfile at the current directory? (y/n)", true)
-              Deliver::DeliverfileCreator.create(enclosed_directory)
-            end
+      Dir.chdir(path) do # switch the context
+        if File.exists?(deliver_path)
+          # Everything looks alright, use the given Deliverfile
+          options.default :beta => false, :skip_deploy => false
+          Deliver::Deliverer.new(deliver_path, force: options.force, is_beta_ipa: options.beta, skip_deploy: options.skip_deploy)
+        else
+          Deliver::Helper.log.warn("No Deliverfile found at path '#{deliver_path}'.")
+          if agree("Do you want to create a new Deliverfile at the current directory? (y/n)", true)
+            Deliver::DeliverfileCreator.create(enclosed_directory)
           end
         end
+      end
     end
 
     command :init do |c|
@@ -70,6 +71,24 @@ class FastlaneApplication
       end
     end
 
+    command :download_screenshots do |c|
+      c.syntax = 'deliver download_screenshots'
+      c.description = "Downloads all existing screenshots from iTunes Connect and stores them in the screenshots folder"
+      c.option '-a', '--app_identifier String', String, 'The App Identifier (e.g. com.krausefx.app)'
+      c.option '-u', '--username String', String, 'Your Apple ID'
+      c.action do |args, options|
+        set_username(options.username)
+
+        app_identifier = options.app_identifier || CredentialsManager::AppfileConfig.try_fetch_value(:app_identifier) || ask("Please enter the app's bundle identifier: ")
+        app = Deliver::App.new(app_identifier: app_identifier)
+
+        path = (Deliver::Helper.fastlane_enabled?? './fastlane' : '.')
+        path = File.join(path, "deliver")
+
+        Deliver::DownloadScreenshots.run(app, path)
+      end
+    end
+
     command :testflight do |c|
       c.syntax = 'deliver testflight'
       c.description = "Uploads a given ipa file to the new Apple TestFlight"

lib/assets/summary.html.erb

@@ -189,7 +189,7 @@
                 deliver couldn't find any screenshots. Due to a bug in iTunes Transporter this will <b>remove</b> the existing screenshots on iTunes Connect.
               </p>
               <p>
-                Please make sure to store your screenshots in the screenshots folder. If you want to download your existing screenshots, run <i>deliver init</i> in a different directory. 
+                Please make sure to store your screenshots in the screenshots folder. If you want to download your existing screenshots, run <i>deliver download_screenshots</i>.
               </p>
             </div>
           <% end %>

lib/deliver/download_screenshots.rb

@@ -0,0 +1,14 @@
+module Deliver
+  class DownloadScreenshots
+    def self.run(app, path)
+      begin
+        Helper.log.info "Downloading all existing screenshots...".green
+        ItunesConnect.new.download_existing_screenshots(app, path)
+        Helper.log.info "Successfully downloaded all existing screenshots".green
+      rescue Exception => ex
+        Helper.log.error ex
+        Helper.log.error "Couldn't download already existing screenshots from iTunesConnect.".red
+      end
+    end
+  end
+end