fastlane 2.212.1 → 2.219.0

data/spaceship/lib/spaceship/connect_api/provisioning/provisioning.rb

@@ -17,8 +17,8 @@ module Spaceship
         # bundleIds
         #
 
-        def get_bundle_ids(filter: {}, includes: nil, limit: nil, sort: nil)
-          params = provisioning_request_client.build_params(filter: filter, includes: includes, limit: limit, sort: sort)
+        def get_bundle_ids(filter: {}, includes: nil, fields: nil, limit: nil, sort: nil)
+          params = provisioning_request_client.build_params(filter: filter, includes: includes, fields: fields, limit: limit, sort: sort)
           provisioning_request_client.get("bundleIds", params)
         end
 
@@ -129,8 +129,8 @@ module Spaceship
         # certificates
         #
 
-        def get_certificates(profile_id: nil, filter: {}, includes: nil, limit: nil, sort: nil)
-          params = provisioning_request_client.build_params(filter: filter, includes: includes, limit: limit, sort: sort)
+        def get_certificates(profile_id: nil, filter: {}, includes: nil, fields: nil, limit: nil, sort: nil)
+          params = provisioning_request_client.build_params(filter: filter, includes: includes, fields: fields, limit: limit, sort: sort)
           if profile_id.nil?
             provisioning_request_client.get("certificates", params)
           else
@@ -164,8 +164,8 @@ module Spaceship
         # devices
         #
 
-        def get_devices(profile_id: nil, filter: {}, includes: nil, limit: nil, sort: nil)
-          params = provisioning_request_client.build_params(filter: filter, includes: includes, limit: limit, sort: sort)
+        def get_devices(profile_id: nil, filter: {}, includes: nil, fields: nil, limit: nil, sort: nil)
+          params = provisioning_request_client.build_params(filter: filter, includes: includes, fields: fields, limit: limit, sort: sort)
           if profile_id.nil?
             provisioning_request_client.get("devices", params)
           else
@@ -213,8 +213,8 @@ module Spaceship
         # profiles
         #
 
-        def get_profiles(filter: {}, includes: nil, limit: nil, sort: nil)
-          params = provisioning_request_client.build_params(filter: filter, includes: includes, limit: limit, sort: sort)
+        def get_profiles(filter: {}, includes: nil, fields: nil, limit: nil, sort: nil)
+          params = provisioning_request_client.build_params(filter: filter, includes: includes, fields: fields, limit: limit, sort: sort)
           provisioning_request_client.get("profiles", params)
         end
 
@@ -253,6 +253,12 @@ module Spaceship
           provisioning_request_client.post("profiles", body)
         end
 
+        def get_profile_bundle_id(profile_id: nil)
+          raise "Profile id is nil" if profile_id.nil?
+
+          provisioning_request_client.get("profiles/#{profile_id}/bundleId")
+        end
+
         def delete_profile(profile_id: nil)
           raise "Profile id is nil" if profile_id.nil?
 

data/spaceship/lib/spaceship/connect_api/testflight/testflight.rb

@@ -204,13 +204,19 @@ module Spaceship
           test_flight_request_client.delete("builds/#{build_id}/relationships/betaGroups", nil, body)
         end
 
-        def create_beta_group(app_id: nil, group_name: nil, is_internal_group: false, public_link_enabled: false, public_link_limit: 10_000, public_link_limit_enabled: false)
+        def create_beta_group(app_id: nil, group_name: nil, is_internal_group: false, public_link_enabled: false, public_link_limit: 10_000, public_link_limit_enabled: false, has_access_to_all_builds: nil)
+          if is_internal_group
+            has_access_to_all_builds = true if has_access_to_all_builds.nil?
+          else
+            # Access to all builds is only for internal groups
+            has_access_to_all_builds = nil
+          end
           body = {
             data: {
               attributes: {
                 name: group_name,
                 isInternalGroup: is_internal_group,
-                hasAccessToAllBuilds: is_internal_group ? true : false, # Undocumented of 2021-08-02 in ASC API docs and ASC Open API spec. This is the default behavior on App Store Connect and does work with both Apple ID and API Token
+                hasAccessToAllBuilds: has_access_to_all_builds, # Undocumented of 2021-08-02 in ASC API docs and ASC Open API spec. This is the default behavior on App Store Connect and does work with both Apple ID and API Token
                 publicLinkEnabled: public_link_enabled,
                 publicLinkLimit: public_link_limit,
                 publicLinkLimitEnabled: public_link_limit_enabled

data/spaceship/lib/spaceship/connect_api/tunes/tunes.rb

@@ -94,7 +94,7 @@ module Spaceship
             }
           end
 
-          app_store_verions_data = platforms.map do |platform|
+          data_for_app_store_versions = platforms.map do |platform|
             {
               type: "appStoreVersions",
               id: "${store-version-#{platform}}"
@@ -103,7 +103,7 @@ module Spaceship
 
           relationships = {
             appStoreVersions: {
-              data: app_store_verions_data
+              data: data_for_app_store_versions
             },
             appInfos: {
               data: [
@@ -160,9 +160,6 @@ module Spaceship
             included << {
               type: "appPrices",
               id: "${price1}",
-              attributes: {
-                startDate: nil
-              },
               relationships: {
                 app: {
                   data: {
@@ -1257,7 +1254,7 @@ module Spaceship
         # resolutionCenter
         #
         # As of 2022-11-11:
-        # This is not official available throught the App Store Connect API using an API Key.
+        # This is not official available through the App Store Connect API using an API Key.
         # This is only works with Apple ID auth.
         #
 

data/spaceship/lib/spaceship/connect_api/users/users.rb

@@ -28,6 +28,36 @@ module Spaceship
           users_request_client.delete("users/#{user_id}")
         end
 
+        # Update existing user
+        def patch_user(user_id:, all_apps_visible:, provisioning_allowed:, roles:, visible_app_ids:)
+          body = {
+            data: {
+              type: 'users',
+              id: user_id,
+              attributes: {
+                allAppsVisible: all_apps_visible,
+                provisioningAllowed: provisioning_allowed,
+                roles: roles
+              },
+              relationships: {
+                visibleApps: {
+                  data: visible_app_ids.map do |app_id|
+                    {
+                      type: "apps",
+                      id: app_id
+                    }
+                  end
+                }
+              }
+            }
+          }
+
+          # Avoid API error: You cannot set visible apps for this user because the user's roles give them access to all apps.
+          body[:data].delete(:relationships) if all_apps_visible
+
+          users_request_client.patch("users/#{user_id}", body)
+        end
+
         # Add app permissions for user
         # @deprecated Use {#post_user_visible_apps} instead.
         def add_user_visible_apps(user_id: nil, app_ids: nil)
@@ -116,6 +146,10 @@ module Spaceship
               }
             }
           }
+
+          # Avoid API error: You cannot set visible apps for this user because the user's roles give them access to all apps.
+          body[:data].delete(:relationships) if all_apps_visible
+
           users_request_client.post("userInvitations", body)
         end
 

data/spaceship/lib/spaceship/connect_api.rb

@@ -76,6 +76,8 @@ require 'spaceship/connect_api/models/actor'
 
 module Spaceship
   class ConnectAPI
+    MAX_OBJECTS_PER_PAGE_LIMIT = 200
+
     # Defined in the App Store Connect API docs:
     # https://developer.apple.com/documentation/appstoreconnectapi/platform
     #

data/spaceship/lib/spaceship/portal/app.rb

@@ -141,7 +141,7 @@ module Spaceship
 
       # Delete this App ID. This action will most likely fail if the App ID is already in the store
       # or there are active profiles
-      # @return (App) The app you just deletd
+      # @return (App) The app you just deleted
       def delete!
         client.delete_app!(app_id, mac: mac?)
         self

data/spaceship/lib/spaceship/portal/app_group.rb

@@ -63,7 +63,7 @@ module Spaceship
       end
 
       # Delete this app group
-      # @return (AppGroup) The app group you just deletd
+      # @return (AppGroup) The app group you just deleted
       def delete!
         client.delete_app_group!(app_group_id)
         self

data/spaceship/lib/spaceship/test_flight/client.rb

@@ -8,7 +8,7 @@ module Spaceship
       #
       # This client is solely responsible for the making HTTP requests and
       # parsing their responses. Parameters should be either named parameters, or
-      # for large request data bodies, pass in anything that can resond to
+      # for large request data bodies, pass in anything that can respond to
       # `to_json`.
       #
       # Each request method should validate the required parameters. A required parameter is one that would result in 400-range response if it is not supplied.

data/spaceship/lib/spaceship/test_flight/tester.rb

@@ -94,7 +94,7 @@ module Spaceship
 
       # @return (Spaceship::TestFlight::Tester) Returns the testers matching the parameter.
       # ITC searches all fields, and is full text. The search results are the union of all words in the search text
-      # @param text (String) (required): Value used to filter the tester, case insensitive
+      # @param text (String) (required): Value used to filter the tester, case-insensitive
       def self.search(app_id: nil, text: nil, is_email_exact_match: false)
         text = text.strip
         testers_matching_text = client.search_for_tester_in_app(app_id: app_id, text: text).map { |data| self.new(data) }

data/spaceship/lib/spaceship/tunes/app_details.rb

@@ -11,7 +11,7 @@ module Spaceship
       # Localized values
       ####
 
-      # @return (Array) Raw access the all available languages. You shouldn't use it probbaly
+      # @return (Array) Raw access the all available languages. You shouldn't use it probably
       attr_accessor :languages
 
       # @return (Hash) A hash representing the app name in all languages
@@ -23,7 +23,7 @@ module Spaceship
       # @return (Hash) A hash representing the privacy URL in all languages
       attr_reader :privacy_url
 
-      # @return (Hash) A hash prepresenting the privacy URL in all languages for Apple TV
+      # @return (Hash) A hash representing the privacy URL in all languages for Apple TV
       attr_reader :apple_tv_privacy_policy
 
       # Categories (e.g. MZGenre.Business)

data/spaceship/lib/spaceship/tunes/app_image.rb

@@ -35,7 +35,7 @@ module Spaceship
       end
 
       def setup
-        # Since September 2015 we don't get the url any more, so we have to manually build it
+        # Since September 2015 we don't get the url anymore, so we have to manually build it
         self.url = "#{HOST_URL}/#{self.asset_token}/0x0ss.jpg"
       end
 

data/spaceship/lib/spaceship/tunes/app_review_attachment.rb

@@ -33,7 +33,7 @@ module Spaceship
       end
 
       def setup
-        # Since September 2015 we don't get the url any more, so we have to manually build it
+        # Since September 2015 we don't get the url anymore, so we have to manually build it
         self.url = "#{HOST_URL}/#{self.asset_token}"
       end
 

data/spaceship/lib/spaceship/tunes/app_submission.rb

@@ -22,7 +22,7 @@ module Spaceship
 
       # To pass from the user
 
-      # @deprecated Setted automatically by <tt>add_id_info_uses_idfa</tt> usage
+      # @deprecated Set automatically by <tt>add_id_info_uses_idfa</tt> usage
       # @return (Boolean) Ad ID Info - Limits ads tracking
       # <b>DEPRECATED:</b> Use <tt>add_id_info_uses_idfa</tt> instead.
       attr_accessor :add_id_info_limits_tracking

data/spaceship/lib/spaceship/tunes/app_version.rb

@@ -78,7 +78,7 @@ module Spaceship
       # @return (Spaceship::Tunes::AppImage) the structure containing information about the large watch icon (1024x1024)
       attr_accessor :watch_app_icon
 
-      # @return (Integer) a unqiue ID for this version generated by App Store Connect
+      # @return (Integer) a unique ID for this version generated by App Store Connect
       attr_accessor :version_id
 
       # @return (Spaceship::Tunes::AppReviewAttachment) the structure containing information about the review attachment file
@@ -147,7 +147,7 @@ module Spaceship
       # @return (String) App Review Information Email Address
       attr_accessor :review_email
 
-      # @return (Boolean) The checkbox that indiciates if a demo account
+      # @return (Boolean) The checkbox that indicates if a demo account
       #   is needed. Is set automatically depending on if a user and pass
       #   are set
       attr_reader :review_user_needed
@@ -546,7 +546,7 @@ module Spaceship
             device_lang_screenshots << new_screenshot
           end
         else # removing
-          raise "cannot remove screenshot with non existing sort_order" unless existing_sort_orders.include?(sort_order)
+          raise "cannot remove screenshot with nonexistent sort_order" unless existing_sort_orders.include?(sort_order)
           device_lang_screenshots.delete_at(existing_sort_orders.index(sort_order))
         end
         setup_screenshots
@@ -624,7 +624,7 @@ module Spaceship
             device_lang_trailers << new_trailer
           end
         else # removing trailer
-          raise "cannot remove trailer with non existing sort_order" unless existing_sort_orders.include?(sort_order)
+          raise "cannot remove trailer with nonexistent sort_order" unless existing_sort_orders.include?(sort_order)
           device_lang_trailers.delete_at(existing_sort_orders.index(sort_order))
         end
         setup_trailers
@@ -754,7 +754,7 @@ module Spaceship
         raise "#{device} isn't a valid device name" unless DisplayFamily.find(device)
 
         languages = raw_data_details.select { |d| d["language"] == language }
-        # IDEA: better error for non existing language
+        # IDEA: better error for nonexistent language
         raise "#{language} isn't an activated language" unless languages.count > 0
         lang_details = languages[0]
         display_families = lang_details["displayFamilies"]["value"]

data/spaceship/lib/spaceship/tunes/build_details.rb

@@ -59,7 +59,7 @@ module Spaceship
 
       # @return (String) Platform of the app
       # @example 'ios'
-      attr_accessor :app_plattform
+      attr_accessor :app_platform
 
       # @return (String) Device Requirements / Device Protocols
       attr_accessor :device_protocols

data/spaceship/lib/spaceship/tunes/iap.rb

@@ -15,14 +15,14 @@ module Spaceship
       # @return (Spaceship::Tunes::Application) A reference to the application
       attr_accessor :application
 
-      # @return (Spaceship::Tunes::IAPFamilies) A reference to the familie list
+      # @return (Spaceship::Tunes::IAPFamilies) A reference to the family list
       def families
         attrs = {}
         attrs[:application] = self.application
         Tunes::IAPFamilies.new(attrs)
       end
 
-      # Creates a new In-App-Purchese on App Store Connect
+      # Creates a new In-App-Purchase on App Store Connect
       # if the In-App-Purchase already exists an exception is raised. Spaceship::TunesClient::ITunesConnectError
       # @param type (String): The Type of the in-app-purchase (Spaceship::Tunes::IAPType::CONSUMABLE,Spaceship::Tunes::IAPType::NON_CONSUMABLE,Spaceship::Tunes::IAPType::RECURRING,Spaceship::Tunes::IAPType::NON_RENEW_SUBSCRIPTION)
       # @param versions (Hash): a Hash of the languages
@@ -49,7 +49,7 @@ module Spaceship
       #      tier: 1
       #    }
       #  ]
-      # @param family_id (String) Only used on RECURRING purchases, assigns the In-App-Purchase to a specific familie
+      # @param family_id (String) Only used on RECURRING purchases, assigns the In-App-Purchase to a specific family
       # @param subscription_free_trial (String) Free Trial duration (1w,1m,3m....)
       # @param subscription_duration (String) 1w,1m.....
       # @param subscription_price_target (Hash) Only used on RECURRING purchases, used to set the

data/spaceship/lib/spaceship/tunes/iap_detail.rb

@@ -220,7 +220,7 @@ module Spaceship
       #   if the iap is not yet cleared for sale
       # @return ([Spaceship::Tunes::PricingInfo]) An array of pricing infos from the same pricing tier
       #   if the iap uses world wide pricing
-      # @return ([Spaceship::Tunes::IAPSubscriptionPricingInfo]) An array of pricing infos from multple subscription pricing tiers
+      # @return ([Spaceship::Tunes::IAPSubscriptionPricingInfo]) An array of pricing infos from multiple subscription pricing tiers
       #   if the iap uses territorial pricing
       def pricing_info
         return [] unless cleared_for_sale
@@ -230,7 +230,7 @@ module Spaceship
 
       private
 
-      # Checks wheather an iap uses world wide or territorial pricing.
+      # Checks whether an iap uses world wide or territorial pricing.
       #
       # @return (true, false)
       def world_wide_pricing?

data/spaceship/lib/spaceship/tunes/iap_families.rb

@@ -13,7 +13,7 @@ module Spaceship
       # @param name (String) Familyname
       # @param product_id (String) New Product's id
       # @param reference_name (String) Reference name of the new product
-      # @param versions (Hash) Localized Familie names
+      # @param versions (Hash) Localized Family names
       # @example
       #  versions: {
       #  'de-DE': {

data/spaceship/lib/spaceship/tunes/iap_family_details.rb

@@ -9,13 +9,13 @@ module Spaceship
       # @return (String) the family name
       attr_accessor :name
 
-      # @return (Intger) the family id
+      # @return (Integer) the family id
       attr_accessor :family_id
 
       # @return (Array) all linked in-app purchases of this family
       attr_accessor :linked_iaps
 
-      # @return (Intger) amount of linked in-app purchases of this family (read-only)
+      # @return (Integer) amount of linked in-app purchases of this family (read-only)
       attr_accessor :iap_count
 
       # @return (Array) of all in-app purchase family details

data/spaceship/lib/spaceship/tunes/iap_family_list.rb

@@ -9,7 +9,7 @@ module Spaceship
       # @return (String) the family name
       attr_accessor :name
 
-      # @return (Intger) the Family Id
+      # @return (Integer) the Family Id
       attr_accessor :family_id
 
       attr_mapping({

data/spaceship/lib/spaceship/tunes/tunes_client.rb

@@ -272,7 +272,7 @@ module Spaceship
       # if it needs to
       #
       # https://github.com/fastlane/fastlane/pull/20480
-      r = request(:get, "https://appstoreconnect.apple.com/iris/v1/apps?include=appStoreVersions,prices")
+      r = request(:get, "https://appstoreconnect.apple.com/iris/v1/apps?include=appStoreVersions")
       response = Spaceship::ConnectAPI::Response.new(
         body: r.body,
         status: r.status,
@@ -659,7 +659,7 @@ module Spaceship
       r = request(:get, "ra/apps/#{app_id}/pricing/intervals")
       data = parse_response(r, 'data')
 
-      # preOrder isn't needed for for the request and has some
+      # preOrder isn't needed for the request and has some
       # values that can cause a failure (invalid dates) so we are removing it
       data.delete('preOrder')
 
@@ -1011,7 +1011,7 @@ module Spaceship
     end
 
     #####################################################
-    # @!group CandiateBuilds
+    # @!group CandidateBuilds
     #####################################################
 
     def candidate_builds(app_id, version_id)

data/supply/lib/supply/client.rb

@@ -78,6 +78,7 @@ module Supply
     private
 
     def call_google_api
+      tries_left ||= (ENV["SUPPLY_UPLOAD_MAX_RETRIES"] || 0).to_i
       yield if block_given?
     rescue Google::Apis::Error => e
       error = begin
@@ -92,7 +93,13 @@ module Supply
         message = e.body
       end
 
-      UI.user_error!("Google Api Error: #{e.message} - #{message}")
+      if tries_left.positive?
+        UI.error("Google Api Error: #{e.message} - #{message} - Retrying...")
+        tries_left -= 1
+        retry
+      else
+        UI.user_error!("Google Api Error: #{e.message} - #{message}")
+      end
     end
   end
 
@@ -305,6 +312,39 @@ module Supply
       return latest_version
     end
 
+    # Get the list of Universal APKs generated by Google from the AAB for a particular version code
+    #
+    # @param [Fixnum] version_code
+    #   Version code of the app bundle.
+    # @raise [FastlaneError] (Google Api Error) If no APK was found for the provided `package_name` and `version_code`
+    # @return [Array<GeneratedUniversalApk>]
+    #
+    def list_generated_universal_apks(package_name:, version_code:)
+      result = call_google_api { client.list_generatedapks(package_name, version_code) }
+
+      result.generated_apks.map do |row|
+        GeneratedUniversalApk.new(package_name, version_code, row.certificate_sha256_hash, row.generated_universal_apk.download_id)
+      end
+    end
+
+    # Download a Universal APK generated by Google for a particular version code
+    #
+    # @param [Supply::GeneratedUniversalApk] generated_universal_apk
+    #   The GeneratedUniversalApk object retrieved from a call to `list_generated_universal_apks`
+    # @param [IO, String] destination
+    #   IO stream or filename to receive content download
+    #
+    def download_generated_universal_apk(generated_universal_apk:, destination:)
+      call_google_api {
+        client.download_generatedapk(
+          generated_universal_apk.package_name,
+          generated_universal_apk.version_code,
+          generated_universal_apk.download_id,
+          download_dest: destination
+        )
+      }
+    end
+
     #####################################################
     # @!group Modifying data
     #####################################################
@@ -499,6 +539,12 @@ module Supply
     # @!group Screenshots
     #####################################################
 
+    # Fetch all the images of a given type and for a given language of your store listing
+    #
+    # @param [String] image_type Typically one of the elements of either Supply::IMAGES_TYPES or Supply::SCREENSHOT_TYPES
+    # @param [String] language Localization code (a BCP-47 language tag; for example, "de-AT" for  Austrian German).
+    # @return [Array<ImageListing>] A list of ImageListing instances describing each image with its id, sha256 and url
+    #
     def fetch_images(image_type: nil, language: nil)
       ensure_active_edit!
 
@@ -511,29 +557,17 @@ module Supply
         )
       end
 
-      urls = (result.images || []).map(&:url)
-      images = urls.map do |url|
-        uri = URI.parse(url)
-        clean_url = [
-          uri.scheme,
-          uri.userinfo,
-          uri.host,
-          uri.port,
-          uri.path
-        ].join
-
-        UI.verbose("Initial URL received: '#{url}'")
-        UI.verbose("Removed params ('#{uri.query}') from the URL")
-        UI.verbose("URL after removing params: '#{clean_url}'")
-
-        full_url = "#{url}=s0" # '=s0' param ensures full image size is returned (https://github.com/fastlane/fastlane/pull/14322#issuecomment-473012462)
-        full_url
+      (result.images || []).map do |row|
+        full_url = "#{row.url}=s0" # '=s0' param ensures full image size is returned (https://github.com/fastlane/fastlane/pull/14322#issuecomment-473012462)
+        ImageListing.new(row.id, row.sha1, row.sha256, full_url)
       end
-
-      return images
     end
 
-    # @param image_type (e.g. phoneScreenshots, sevenInchScreenshots, ...)
+    # Upload an image or screenshot of a specific type for a given language to your store listing
+    #
+    # @param [String] image_type (e.g. phoneScreenshots, sevenInchScreenshots, ...)
+    # @param [String] language localization code (i.e. BCP-47 language tag as in `pt-BR`)
+    #
     def upload_image(image_path: nil, image_type: nil, language: nil)
       ensure_active_edit!
 
@@ -549,6 +583,11 @@ module Supply
       end
     end
 
+    # Remove all screenshots of a given image_type and language from the store listing
+    #
+    # @param [String] image_type (e.g. phoneScreenshots, sevenInchScreenshots, ...)
+    # @param [String] language localization code (i.e. BCP-47 language tag as in `pt-BR`)
+    #
     def clear_screenshots(image_type: nil, language: nil)
       ensure_active_edit!
 
@@ -562,6 +601,26 @@ module Supply
       end
     end
 
+    # Remove a specific screenshot of a given image_type and language from the store listing
+    #
+    # @param [String] image_type (e.g. phoneScreenshots, sevenInchScreenshots, ...)
+    # @param [String] language localization code (i.e. BCP-47 language tag as in `pt-BR`)
+    # @param [String] image_id The id of the screenshot to remove (as per `ImageListing#id`)
+    #
+    def clear_screenshot(image_type: nil, language: nil, image_id: nil)
+      ensure_active_edit!
+
+      call_google_api do
+        client.delete_edit_image(
+          current_package_name,
+          current_edit.id,
+          language,
+          image_type,
+          image_id
+        )
+      end
+    end
+
     def upload_obb(obb_file_path: nil, apk_version_code: nil, expansion_file_type: nil)
       ensure_active_edit!
 

data/supply/lib/supply/generated_universal_apk.rb

@@ -0,0 +1,24 @@
+module Supply
+  # A model representing the returned values from a call to Client#list_generated_universal_apks
+  class GeneratedUniversalApk
+    attr_accessor :package_name
+    attr_accessor :version_code
+    attr_accessor :certificate_sha256_hash
+    attr_accessor :download_id
+
+    # Initializes the Generated Universal APK model
+    def initialize(package_name, version_code, certificate_sha256_hash, download_id)
+      self.package_name = package_name
+      self.version_code = version_code
+      self.certificate_sha256_hash = certificate_sha256_hash
+      self.download_id = download_id
+    end
+
+    def ==(other)
+      self.package_name == other.package_name \
+        && self.version_code == other.version_code \
+        && self.certificate_sha256_hash == other.certificate_sha256_hash \
+        && self.download_id == other.download_id
+    end
+  end
+end

data/supply/lib/supply/image_listing.rb

@@ -0,0 +1,15 @@
+module Supply
+  class ImageListing
+    attr_reader :id
+    attr_reader :sha1
+    attr_reader :sha256
+    attr_reader :url
+
+    def initialize(id, sha1, sha256, url)
+      @id = id
+      @sha1 = sha1
+      @sha256 = sha256
+      @url = url
+    end
+  end
+end

data/supply/lib/supply/options.rb

@@ -210,6 +210,11 @@ module Supply
                                      description: "Whether to skip uploading SCREENSHOTS",
                                      type: Boolean,
                                      default_value: false),
+        FastlaneCore::ConfigItem.new(key: :sync_image_upload,
+                                     env_name: "SUPPLY_SYNC_IMAGE_UPLOAD",
+                                     description: "Whether to use sha256 comparison to skip upload of images and screenshots that are already in Play Store",
+                                     type: Boolean,
+                                     default_value: false),
         FastlaneCore::ConfigItem.new(key: :track_promote_to,
                                      env_name: "SUPPLY_TRACK_PROMOTE_TO",
                                      optional: true,

data/supply/lib/supply/setup.rb

@@ -66,14 +66,14 @@ module Supply
 
           UI.message("Downloading `#{image_type}` for #{listing.language}...")
 
-          urls = client.fetch_images(image_type: image_type, language: listing.language)
+          urls = client.fetch_images(image_type: image_type, language: listing.language).map(&:url)
           next if urls.nil? || urls.empty?
 
           image_counter = 1 # Used to prefix the downloaded files, so order is preserved.
           urls.each do |url|
             if IMAGES_TYPES.include?(image_type) # IMAGE_TYPES are stored in locale/images
               file_path = "#{path}.#{FastImage.type(url)}"
-            else # SCREENSHOT_TYPES are stored in locale/images/<screensho_types>
+            else # SCREENSHOT_TYPES are stored in locale/images/<screenshot_types>
               file_path = File.join(path, "#{image_counter}_#{listing.language}.#{FastImage.type(url)}")
             end