eco-helpers 1.3.13 → 1.3.18

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5b59fa190429f792010af645ab28370ef0132ad17fafca035a363722736ab01e
-  data.tar.gz: c35a11ee4935356f1a93a008b63b520bd0809ef3a3d12d2b71e5329eb9f721f8
+  metadata.gz: a61c411db84db417e4d9004d2e8f9359b423cb91ac31b40eaf390e9b3e7a4f0a
+  data.tar.gz: 3431111bd4259b33a8c6ac38ad5ce67226c4e2a8c9c95b311142d7896f9d68f1
 SHA512:
-  metadata.gz: e9d7f407038348bfb98bc8c1cf127d73108f3e90b92d3cd267f8c1868a1fad5810226506e4112f7c87d9deb6775af2d7ad94bb27dfd4c6c2ea2885b28f7b6c76
-  data.tar.gz: 90c6d245f2eb3fbfa1729765cf7629792eadbd33b84c58333aaac6a7e6a81d395fa150c4ff9c240c24403976c25d471d510441b849159aeaf67a593c19258db1
+  metadata.gz: afae206900769f68a8733dbe72d3277cf610fd4d7e92a1adda5197685889511fb80c9a424987f59306c59fa251b79a6f929d0dfc566c301cd3703c5fc6c5a2bf
+  data.tar.gz: 19d0d4c7c3a24c194b02faee98bd2ad2c55d8a8d7522681f2d1918594207cf673b827e4a510646c1f21b302105290b7b31da41be98a80112ff6d8c0c6e398826

data/CHANGELOG.md

@@ -1,6 +1,50 @@
 # Change Log
 All notable changes to this project will be documented in this file.
 
+## [1.3.18] - 2020-07-xx
+
+### Added
+### Changed
+### Fixed
+  - the `update` case was missing the code to use the `default_usergroup`
+
+
+## [1.3.17] - 2020-07-06
+
+### Added
+### Changed
+  - the `hris` case should not only include as `leavers` those that have account, but anyone that leaves
+    * as we could have active people with no account
+### Fixed
+  - `Eco::API::Session::Batch::Errors#errors`: fix error for empty `body` on response
+
+## [1.3.16] - 2020-06-26
+
+### Added
+  - `Eco::API::Common::People::Entries`, `#entry` & `#find` should allow for `strict` search (added parameter)
+### Changed
+  - upgraded `ecoportal-api` gem dependency to minimum version `0.5.7`
+    - this includes the removal of some patches
+  - some gems update
+### Fixed
+  - patched preferences (`kiosk` preferences should default to `nil`)
+
+## [1.3.15] - 2020-06-11
+
+### Added
+  - default usecase to export to `csv` (`-detailed`) now includes `"Supervisor Name"` column
+### Changed
+### Fixed
+
+## [1.3.14] - 2020-06-10
+
+### Added
+  - `Eco::API::Common::People::SupervisorHelpers` now has its methods as class methods
+### Changed
+  - upgraded `ecoportal-api` gem dependency to minimum version `0.5.6`
+### Fixed
+
+
 ## [1.3.13] - 2020-05-29
 
 ### Added

data/eco-helpers.gemspec

@@ -22,16 +22,16 @@ Gem::Specification.new do |s|
   #s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
   s.require_paths = ["lib"]
 
-  s.add_development_dependency "bundler",   "~> 2.0.1"
-  s.add_development_dependency "rspec",     "~> 3",   ">= 3.8"
+  s.add_development_dependency "bundler",   "~> 2.1", ">= 2.1.3"
+  s.add_development_dependency "rspec",     "~> 3",   ">= 3.9"
   s.add_development_dependency "rake",      "~> 12.3.0"
   s.add_development_dependency "yard",      "~> 0.9", ">= 0.9.18"
   s.add_development_dependency "redcarpet", "~> 3.5", ">= 3.5.0"
 
-  s.add_dependency 'ecoportal-api', '~> 0.5',  '>= 0.5.5'
-  s.add_dependency 'aws-sdk-s3',    '~> 1.30', '>= 1.30.1'
-  s.add_dependency 'aws-sdk-ses',   '~> 1.14', '>= 1.14.0'
-  s.add_dependency 'dotenv',        '~> 2.6',  '>= 2.6.0'
+  s.add_dependency 'ecoportal-api', '~> 0.5',  '>= 0.5.8'
+  s.add_dependency 'aws-sdk-s3',    '~> 1',    '>= 1.64.0'
+  s.add_dependency 'aws-sdk-ses',   '~> 1',    '>= 1.29.0'
+  s.add_dependency 'dotenv',        '~> 2.7',  '>= 2.7.0'
   s.add_dependency 'net-sftp',      '~> 2.1',  '>= 2.1.2'
 
 end

data/lib/eco/api/common/people/entries.rb

@@ -50,23 +50,28 @@ module Eco
           end
 
           # Search function to find an `entry` based on one of different options
-          def entry(id: nil, external_id: nil, email: nil)
+          def entry(id: nil, external_id: nil, email: nil, strict: false)
             init_caches
             pers = nil
             pers = @by_id[id]&.first if id
             pers = @by_external_id[external_id&.strip]&.first    if !pers && !external_id.to_s.strip.empty?
-            pers = @by_email[email&.downcase.strip]&.first       if !pers && !email.to_s.strip.empty?
-            pers = @by_external_id[email&.downcase.strip]&.first if !pers && !email.to_s.strip.empty?
+
+            # strict prevents taking existing user for searched person with same email
+            # specially useful if the organisation ensures all have external id (no need for email search)
+            if !pers && (!strict || external_id.to_s.strip.empty?)
+              pers = @by_email[email&.downcase.strip]&.first       if !pers && !email.to_s.strip.empty?
+              pers = @by_external_id[email&.downcase.strip]&.first if !pers && !email.to_s.strip.empty?
+            end
             pers
           end
 
           # Search function to find an `entry` based on one of different options
           # see Eco::API::Common::People::Entries#entry
-          def find(object)
+          def find(object, strict: false)
             id          = object.respond_to?("id")?          object.send("id") : nil
             external_id = object.respond_to?("external_id")? object.send("external_id") : nil
             email       = object.respond_to?("email")?       object.send("email") : nil
-            entry(id: id, external_id: external_id, email: email)
+            entry(id: id, external_id: external_id, email: email, strict: strict)
           end
 
           def exclude(object)

data/lib/eco/api/common/people/supervisor_helpers.rb

@@ -7,134 +7,144 @@ module Eco
         #   * `external_id`
         #   * `id`
         module SupervisorHelpers
+          def self.included(base)
+            base.send(:include, ClassMethods)
+          end
 
-          # Reorders as follows:
-          #   1. supervisors, people with no supervisor or where their supervisor not present
-          #   2. subordinates
-          def sort_by_supervisors(values, supervisors_first: true)
-            raise "Expected non hash Enumerable. Given: #{values.class}" if values.is_a?(Hash)
-            return [] unless values && values.is_a?(Enumerable)
-            roam = Proc.new do |tree|
-              [].tap do |out|
-                sub_outs = tree.empty?? [] : tree.map {|sup, subtree| roam.call(subtree)}
-                tree.each do |sup, subtree|
-                  sout = subtree.empty?? [] :roam.call(subtree)
-                  supervisors_first ? sout.unshift(sup) : sout.push(sup)
-                  out.concat(sout)
+          module ClassMethods
+            # Reorders as follows:
+            #   1. supervisors, people with no supervisor or where their supervisor not present
+            #   2. subordinates
+            def sort_by_supervisors(values, supervisors_first: true)
+              raise "Expected non hash Enumerable. Given: #{values.class}" if values.is_a?(Hash)
+              return [] unless values && values.is_a?(Enumerable)
+              roam = Proc.new do |tree|
+                [].tap do |out|
+                  sub_outs = tree.empty?? [] : tree.map {|sup, subtree| roam.call(subtree)}
+                  tree.each do |sup, subtree|
+                    sout = subtree.empty?? [] :roam.call(subtree)
+                    supervisors_first ? sout.unshift(sup) : sout.push(sup)
+                    out.concat(sout)
+                  end
                 end
               end
-            end
 
-            roam.call(supervisors_tree(values))
-          end
+              roam.call(supervisors_tree(values))
+            end
 
-          def tree_to_str(tree, lev: 0)
-            raise "Required Hash tree structure. Given: #{tree.class}" unless tree.is_a?(Hash)
-            "".tap do |str|
-              tree.each do |entry, subtree|
-                str << "#{"  " * lev}+-- #{entry.id || entry.external_id}\n"
-                str << tree_to_str(subtree, lev: lev + 1) unless !subtree || subtree.empty?
+            def tree_to_str(tree, lev: 0)
+              raise "Required Hash tree structure. Given: #{tree.class}" unless tree.is_a?(Hash)
+              "".tap do |str|
+                tree.each do |entry, subtree|
+                  str << "#{"  " * lev}+-- #{entry.id || entry.external_id}\n"
+                  str << tree_to_str(subtree, lev: lev + 1) unless !subtree || subtree.empty?
+                end
               end
             end
-          end
 
-          def print_tree(tree, lev: 0)
-            puts tree_to_str(tree)
-          end
+            def print_tree(tree, lev: 0)
+              puts tree_to_str(tree)
+            end
 
-          # Generates a `Hash` tree structure, where:
-          #   * **keys** are nodes
-          #   * **values** are `Hash` subtree structures of `key` subordinates
-          # @note it is resilient to cyclic supervisors (it will just add the last at the top)
-          # @param values [Enumerable<Object>] of objects with methods:
-          #  `id`, `external_id` and `supervisor_id`
-          # @return [Hash] the tree structure
-          def supervisors_tree(values)
-            raise "Expected non hash Enumerable. Given: #{values.class}" if values.is_a?(Hash)
-            return {} unless values && values.is_a?(Enumerable)
-            idx = get_super_indexes(values)
-
-            processed = []
-            subtree = Proc.new do |entry, level, toptree|
-              if processed.include?(entry)
-                next {} unless toptree.key?(entry) && level > 0
-                # needs to be moved as a child
-                subnodes = toptree.delete(entry)
-                processed.delete(entry)
-              end
+            # Generates a `Hash` tree structure, where:
+            #   * **keys** are nodes
+            #   * **values** are `Hash` subtree structures of `key` subordinates
+            # @note it is resilient to cyclic supervisors (it will just add the last at the top)
+            # @param values [Enumerable<Object>] of objects with methods:
+            #  `id`, `external_id` and `supervisor_id`
+            # @return [Hash] the tree structure
+            def supervisors_tree(values)
+              raise "Expected non hash Enumerable. Given: #{values.class}" if values.is_a?(Hash)
+              return {} unless values && values.is_a?(Enumerable)
+              idx = get_super_indexes(values)
+
+              processed = []
+              subtree = Proc.new do |entry, level, toptree|
+                if processed.include?(entry)
+                  next {} unless toptree.key?(entry) && level > 0
+                  # needs to be moved as a child
+                  subnodes = toptree.delete(entry)
+                  processed.delete(entry)
+                end
 
-              subnodes ||= {}.tap do |tree|
-                subs = idx[:subordinates].call(entry)
-                processed.push(entry)
-                next nil unless subs && !subs.empty?
-                subs.each do |sub|
-                  sub_tree = subtree.call(sub, level + 1, toptree)
-                  tree.merge!(sub_tree)
+                subnodes ||= {}.tap do |tree|
+                  subs = idx[:subordinates].call(entry)
+                  processed.push(entry)
+                  next nil unless subs && !subs.empty?
+                  subs.each do |sub|
+                    sub_tree = subtree.call(sub, level + 1, toptree)
+                    tree.merge!(sub_tree)
+                  end
                 end
-              end
 
-              {entry => subnodes}
-            end
+                {entry => subnodes}
+              end
 
-            {}.tap do |tree|
-              idx[:by_sup].keys.each do |sup_id|
-                if sup = idx[:supers][sup_id]
-                  tree.merge!(subtree.call(sup, 0, tree))
-                else
-                  idx[:by_sup][sup_id].each do |sub|
-                    tree.merge!(subtree.call(sub, 0, tree))
+              {}.tap do |tree|
+                idx[:by_sup].keys.each do |sup_id|
+                  if sup = idx[:supers][sup_id]
+                    tree.merge!(subtree.call(sup, 0, tree))
+                  else
+                    idx[:by_sup][sup_id].each do |sub|
+                      tree.merge!(subtree.call(sub, 0, tree))
+                    end
                   end
                 end
               end
             end
-          end
 
-          private
-
-          def get_super_indexes(values)
-            raise "Expected non hash Enumerable. Given: #{values.class}" if values.is_a?(Hash)
-            {}.tap do |indexes|
-              indexes[:by_id]  = values.map do |e|
-                e.id && [e.id, e]
-              end.compact.to_h
-              indexes[:by_ext] = values.map do |e|
-                e.external_id && [e.external_id, e]
-              end.compact.to_h
-
-              indexes[:by_sup] = {}.tap do |by_s|
-                values.group_by do |e|
-                  e.supervisor_id
-                end.tap do |by_sup|
-                  by_s[nil] = by_sup.delete(nil) if by_sup.key?(nil)
-                  by_s.merge!(by_sup)
+            private
+
+            def get_super_indexes(values)
+              raise "Expected non hash Enumerable. Given: #{values.class}" if values.is_a?(Hash)
+              {}.tap do |indexes|
+                indexes[:by_id]  = values.map do |e|
+                  e.id && [e.id, e]
+                end.compact.to_h
+                indexes[:by_ext] = values.map do |e|
+                  e.external_id && [e.external_id, e]
+                end.compact.to_h
+
+                indexes[:by_sup] = {}.tap do |by_s|
+                  values.group_by do |e|
+                    e.supervisor_id
+                  end.tap do |by_sup|
+                    by_s[nil] = by_sup.delete(nil) if by_sup.key?(nil)
+                    by_s.merge!(by_sup)
+                  end
                 end
-              end
 
-              indexes[:supers] = {}.tap do |sups|
-                indexes[:by_ext].select do |ext, e|
-                  ext && indexes[:by_sup].key?(ext)
-                end.tap {|found| sups.merge!(found)}
-                indexes[:by_id].select do |id, e|
-                  id && indexes[:by_sup].key?(id)
-                end.tap {|found| sups.merge!(found)}
-              end
+                indexes[:supers] = {}.tap do |sups|
+                  indexes[:by_ext].select do |ext, e|
+                    ext && indexes[:by_sup].key?(ext)
+                  end.tap {|found| sups.merge!(found)}
+                  indexes[:by_id].select do |id, e|
+                    id && indexes[:by_sup].key?(id)
+                  end.tap {|found| sups.merge!(found)}
+                end
 
-              indexes[:is_super]     = Proc.new do |entry|
-                !!(indexes[:supers][entry.id] || indexes[:supers][entry.external_id])
-              end
+                indexes[:is_super]     = Proc.new do |entry|
+                  !!(indexes[:supers][entry.id] || indexes[:supers][entry.external_id])
+                end
 
-              indexes[:subordinates] = Proc.new do |entry|
-                subs = nil
-                if sup = indexes[:supers][entry.id] || indexes[:supers][entry.external_id]
-                  subs ||= indexes[:by_sup][sup.id]          unless !sup.id
-                  subs ||= indexes[:by_sup][sup.external_id] unless !sup.external_id
+                indexes[:subordinates] = Proc.new do |entry|
+                  subs = nil
+                  if sup = indexes[:supers][entry.id] || indexes[:supers][entry.external_id]
+                    subs ||= indexes[:by_sup][sup.id]          unless !sup.id
+                    subs ||= indexes[:by_sup][sup.external_id] unless !sup.external_id
+                  end
+                  subs
                 end
-                subs
               end
+
             end
 
           end
 
+          class << self
+            include SupervisorHelpers::ClassMethods
+          end
+
         end
       end
     end

data/lib/eco/api/common/version_patches/ecoportal_api.rb

@@ -1,6 +1,4 @@
-require 'ecoportal/api'
 require_relative 'ecoportal_api/base_model'
-require_relative 'ecoportal_api/external_people'
 require_relative 'ecoportal_api/external_person'
 require_relative 'ecoportal_api/internal_person'
 require_relative 'ecoportal_api/account_preferences'

data/lib/eco/api/common/version_patches/ecoportal_api/account_preferences.rb

@@ -1,47 +1,28 @@
 module Ecoportal
   module API
     class Internal
-      class Preferences
+      class Preferences < Common::BaseModel
+        passthrough  :kiosk_enabled, :kiosk_workflow_message, :kiosk_create_button_label,
+            :kiosk_create_button_help, :kiosk_return_button_label, :kiosk_return_button_help,
+            :kiosk_dashboard_button_label, :kiosk_dashboard_button_help
+
         [
           :show_sidebar,
           :show_shortcuts,
           :show_coming_soon,
           :show_recently_visited_forms,
           :show_tasks,
-        ].each do |field|
-          key = field.to_s
+          :show_task_bubbles
+        ].map(&:to_s).each do |field|
           define_method(field) do
-            if doc.key?(key)
-              doc[key]
+            if doc.key?(field)
+              doc[field]
             else
               true
             end
           end
           define_method("#{field}=") do |value|
-            doc[key] = !!value
-          end
-        end
-
-        [
-          :kiosk_enabled,
-          :kiosk_workflow_message,
-          :kiosk_create_button_label,
-          :kiosk_create_button_help,
-          :kiosk_return_button_label,
-          :kiosk_return_button_help,
-          :kiosk_dashboard_button_label,
-          :kiosk_dashboard_button_help
-        ].each do |field|
-          key = field.to_s
-          define_method(field) do
-            if doc.key?(key)
-              doc[key]
-            else
-              nil
-            end
-          end
-          define_method("#{field}=") do |value|
-            doc[key] = !!value
+            doc[field] = !!value
           end
         end
 

data/lib/eco/api/session/batch/errors.rb

@@ -69,13 +69,15 @@ module Eco
 
           def errors
             entries.each_with_object([]) do |entry, arr|
-              if errs = status[entry].body["errors"]
-                errs.each do |msg|
-                  arr.push({
-                    type:  klass = Eco::API::Error.get_type(msg),
-                    err:   klass.new(err_msg: msg, entry: entry, session: session),
-                    entry: entry
-                  })
+              if body = status[entry].body
+                if errs = body["errors"]
+                  errs.each do |msg|
+                    arr.push({
+                      type:  klass = Eco::API::Error.get_type(msg),
+                      err:   klass.new(err_msg: msg, entry: entry, session: session),
+                      entry: entry
+                    })
+                  end
                 end
               end
             end

data/lib/eco/api/usecases/default_cases/create_case.rb

@@ -69,7 +69,7 @@ module Eco
                     if session.tagtree
                       person.account.default_tag = session.tagtree.default_tag(*person.filter_tags)
                     else
-                      tags = person.filter_tags
+                      tags = person.filter_tags || []
                       person.account.default_tag = tags.first unless tags.length > 1
                     end
                   end

data/lib/eco/api/usecases/default_cases/hris_case.rb

@@ -11,17 +11,18 @@ module Eco
               supers   = session.job_group("post").new("supers", usecase: usecase, type: :update, sets: :core)
               leavers  = session.job_group("post").new("leavers", usecase: usecase, type: :update, sets: :account)
 
-              people.users.each_with_index  do |person, i|
-                if !entries.find(person)
+              strict_search = session.config.people.strict_search? && (!options[:search]&.key?(:strict) || options.dig(:search, :strict))
+              pgs           = session.policy_groups
+
+              people.each_with_index  do |person, i|
+                if !entries.find(person, strict: strict_search)
                   leavers.add(person) do |person|
                     person.supervisor_id = nil
-                    person.account       = nil
+                    person.account       = nil if person.account
                   end
                 end
               end
 
-              strict_search = session.config.people.strict_search? && (!options[:search]&.key?(:strict) || options.dig(:search, :strict))
-              pgs           = session.policy_groups
 
               if session.config.people.default_usergroup?
                 def_id  = pgs.to_id(session.config.people.default_usergroup)
@@ -84,7 +85,7 @@ module Eco
                     if session.tagtree
                       person.account.default_tag = session.tagtree.default_tag(*person.filter_tags)
                     else
-                      tags = person.filter_tags
+                      tags = person.filter_tags || []
                       person.account.default_tag = tags.first unless tags.length > 1
                     end
                   end

data/lib/eco/api/usecases/default_cases/set_default_tag_case.rb

@@ -30,8 +30,12 @@ module Eco
               update   = session.job_group("main").new("update", usecase: usecase, type: :update, sets: :account)
 
               people.each_with_index  do |person, i|
-                #next unless person.id == "5c527ba63f7690001243f5b2"
-                person.account.default_tag = session.tagtree.default_tag(*person.filter_tags)
+                if session.tagtree
+                  person.account.default_tag = session.tagtree.default_tag(*person.filter_tags)
+                else
+                  tags = person.filter_tags || []
+                  person.account.default_tag = tags.first unless tags.length > 1
+                end
                 update.add(person)
               end
 

data/lib/eco/api/usecases/default_cases/to_csv_case.rb

@@ -42,7 +42,7 @@ module Eco
                   csv << session.new_entry(person, dependencies: deps).to_hash.values
                 end
               end
-              true
+              exit(0)
             end
           end
 

data/lib/eco/api/usecases/default_cases/to_csv_detailed_case.rb

@@ -24,21 +24,31 @@ module Eco
               CSV.open(file, "w") do |csv|
                 deps    = {"supervisor_id" => {people: people}}
                 header  = session.new_entry(people.first, dependencies: deps).to_hash.keys
+                header += ["Subordinates"]
+                header += ["Supervisor Name"]
                 header += abilities
                 header += ["Login Methods"] if login_providers.any_optional?
-                header += ["Subordinates"]
                 header += ["Landing Page"]
 
                 csv << header
                 people.each do |person|
                   data = session.new_entry(person, dependencies: deps).to_hash.values
+
+                  data.push(person.subordinates)
+                  super_id = person.supervisor_id
+                  if supervisor = people.person(id: super_id, external_id: super_id)
+                    data.push(supervisor.name)
+                  else
+                    data.push("")
+                  end
+
                   person_abilities = (person.account && person.account.permissions_custom) || {}
                   data += abilities.map {|key| person_abilities[key] || "no access"}
+
                   if login_providers.any_optional?
                     logins = (person.account && person.account.login_provider_ids) || []
                     data.push(login_providers.to_name(logins).join("|"))
                   end
-                  data.push(person.subordinates)
 
                   if person.account && landing_id = person.account.landing_page_id
                     data.push(landing_id)
@@ -47,7 +57,7 @@ module Eco
                   csv << data
                 end
               end
-              true
+              exit(0)
             end
           end
 

data/lib/eco/api/usecases/default_cases/update_case.rb

@@ -12,6 +12,10 @@ module Eco
               strict_search = session.config.people.strict_search? && (!options[:search]&.key?(:strict) || options.dig(:search, :strict))
               pgs           = session.policy_groups
 
+              if session.config.people.default_usergroup?
+                def_id  = pgs.to_id(session.config.people.default_usergroup)
+              end
+
               entries.each.with_index do |entry, i|
                 if person = people.find(entry, strict: strict_search)
 
@@ -37,6 +41,7 @@ module Eco
                   entry.set_details(person) unless options.dig(:exclude, :details)
 
                   unless options.dig(:exclude, :account)
+                    add_account = !person.account
                     ini_pg_ids = person.account&.policy_group_ids || []
 
                     account_excluded = []
@@ -68,7 +73,7 @@ module Eco
                       if session.tagtree
                         person.account.default_tag = session.tagtree.default_tag(*person.filter_tags)
                       else
-                        tags = person.filter_tags
+                        tags = person.filter_tags || []
                         person.account.default_tag = tags.first unless tags.length > 1
                       end
                     end

data/lib/eco/api/usecases/default_cases/upsert_case.rb

@@ -77,7 +77,7 @@ module Eco
                     if session.tagtree
                       person.account.default_tag = session.tagtree.default_tag(*person.filter_tags)
                     else
-                      tags = person.filter_tags
+                      tags = person.filter_tags || []
                       person.account.default_tag = tags.first unless tags.length > 1
                     end
                   end

data/lib/eco/version.rb

@@ -1,3 +1,3 @@
 module Eco
-  VERSION = "1.3.13"
+  VERSION = "1.3.18"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: eco-helpers
 version: !ruby/object:Gem::Version
-  version: 1.3.13
+  version: 1.3.18
 platform: ruby
 authors:
 - Oscar Segura
@@ -16,14 +16,20 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.0.1
+        version: '2.1'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.1.3
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.0.1
+        version: '2.1'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.1.3
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -33,7 +39,7 @@ dependencies:
         version: '3'
     - - ">="
       - !ruby/object:Gem::Version
-        version: '3.8'
+        version: '3.9'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -43,7 +49,7 @@ dependencies:
         version: '3'
     - - ">="
       - !ruby/object:Gem::Version
-        version: '3.8'
+        version: '3.9'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -107,7 +113,7 @@ dependencies:
         version: '0.5'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.5.5
+        version: 0.5.8
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -117,67 +123,67 @@ dependencies:
         version: '0.5'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.5.5
+        version: 0.5.8
 - !ruby/object:Gem::Dependency
   name: aws-sdk-s3
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.30'
+        version: '1'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.30.1
+        version: 1.64.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.30'
+        version: '1'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.30.1
+        version: 1.64.0
 - !ruby/object:Gem::Dependency
   name: aws-sdk-ses
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 1.14.0
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.14'
+        version: '1'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.29.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 1.14.0
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.14'
+        version: '1'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.29.0
 - !ruby/object:Gem::Dependency
   name: dotenv
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 2.6.0
+        version: 2.7.0
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.6'
+        version: '2.7'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 2.6.0
+        version: 2.7.0
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.6'
+        version: '2.7'
 - !ruby/object:Gem::Dependency
   name: net-sftp
   requirement: !ruby/object:Gem::Requirement
@@ -249,7 +255,6 @@ files:
 - lib/eco/api/common/version_patches/ecoportal_api.rb
 - lib/eco/api/common/version_patches/ecoportal_api/account_preferences.rb
 - lib/eco/api/common/version_patches/ecoportal_api/base_model.rb
-- lib/eco/api/common/version_patches/ecoportal_api/external_people.rb
 - lib/eco/api/common/version_patches/ecoportal_api/external_person.rb
 - lib/eco/api/common/version_patches/ecoportal_api/internal_person.rb
 - lib/eco/api/common/version_patches/exception.rb

data/lib/eco/api/common/version_patches/ecoportal_api/external_people.rb

@@ -1,38 +0,0 @@
-module Ecoportal
-  module API
-    class V1
-      class People
-
-        def job
-          operation = Common::BatchOperation.new("/people", person_class, logger: client.logger)
-          yield operation
-          # The batch operation is responsible for logging the output
-          job_id = create_job(operation)
-          status = wait_for_job_completion(job_id)
-
-          if status&.complete?
-            operation.process_response job_result(job_id, operation)
-          else
-            raise "Job `#{job_id}` not complete. Probably timeout after #{JOB_TIMEOUT + 120} seconds. Current status: #{status}"
-          end
-        end
-
-        private
-
-        def wait_for_job_completion(job_id)
-          # timeout library is evil. So we make poor-man timeout.
-          # https://jvns.ca/blog/2015/11/27/why-rubys-timeout-is-dangerous-and-thread-dot-raise-is-terrifying/
-          before = Time.now
-          while true
-            status = job_status(job_id)
-            break status if status.complete?
-            break status if Time.now >= before + (JOB_TIMEOUT + 120)
-            sleep(DELAY_STATUS_CHECK)
-            status
-          end
-        end
-
-      end
-    end
-  end
-end