servizehub 1.0.1 → 1.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 273b3e612c965437c44f14fd30236e43090b8ffebe5bffee90f14716820cd07e
-  data.tar.gz: 0f9b3edcc27a9bfb83170026ec1a09ccd3e3abbe034b5c326b6d6afe545d3bcb
+  metadata.gz: fa03e8c0c5f26ea1c781e58515d2c829a6703fffeec1fa7f9b4653b67dc32943
+  data.tar.gz: 48d3161c5c993ce33b114e604306ca26f90aa0c7dad0b4af6bdb62688eaa8c2e
 SHA512:
-  metadata.gz: 5a672d24f9f8574dc52f6d4cc36159486a1ccf8ef6f687c4bbe590e1b7877da87b8b7df3cbb2c2781cba75084651a6672d7eca548dd74f482a16f278a4e2eeac
-  data.tar.gz: fcdda5248e398fe5520fa3a35c747b74d7272d8940395dc0e45039e1a4596eedab13991611037f66111a3bb0f31ae3e30e2d73179552475d30ca90540956e415
+  metadata.gz: 00a65a937a2b0a8013211ca48fe060a6a59a4e3e60bf2b3133a2dd3c7cd16808512e8a9782889f27cd650da9a1e63a27960ef6be01371d9706182b2fcda8618b
+  data.tar.gz: 2369cf93af7ff906b1ca1993dab07fbd0cef1bcd38c01450ca12c4d17d3c32f36c18d4c9bdcc983aa7094b207bc15d01e6341f3b4afb62656a6d2c35d2977728

data/lib/servizehub/client.rb

@@ -12,7 +12,7 @@ module Servizehub
       @api_key = api_key
     end
 
-    # Helper to validate date format YYYY-MM-DD
+    # Validate date format YYYY-MM-DD
     def valid_date?(date)
       Date.strptime(date, '%Y-%m-%d')
       true
@@ -21,79 +21,80 @@ module Servizehub
     end
 
     def send_booking(date:, service_type:, status:)
-      raise ArgumentError, "All booking details are required" if date.nil? || service_type.nil? || status.nil?
-      raise ArgumentError, "Invalid date format, expected YYYY-MM-DD" unless valid_date?(date)
+      return "All booking details are required" if date.nil? || service_type.nil? || status.nil?
+      return "Invalid date format, expected YYYY-MM-DD" unless valid_date?(date)
 
       valid_statuses = ["available", "unavailable"]
-      raise ArgumentError, "Invalid status value" unless valid_statuses.include?(status)
+      return "Invalid status value" unless valid_statuses.include?(status)
 
       uri = URI.parse("#{BASE_URL}/external/capture-availability")
+
       request = Net::HTTP::Post.new(uri)
       request["Content-Type"] = "application/json"
       request["servizhub-api-key"] = @api_key
+
       request.body = {
         date: date,
         serviceType: service_type,
         status: status
       }.to_json
 
-      response = Net::HTTP.start(uri.hostname, uri.port, use_ssl: uri.scheme == "https") do |http|
+      response = Net::HTTP.start(
+        uri.hostname,
+        uri.port,
+        use_ssl: uri.scheme == "https"
+      ) do |http|
         http.request(request)
       end
 
-      unless response.code.to_i < 400
-        begin
-          data = JSON.parse(response.body)
-          message = data["message"] || "Booking failed"
-        rescue
-          message = "Booking failed"
-        end
-        raise message
+      begin
+        data = JSON.parse(response.body)
+      rescue JSON::ParserError
+        data = { "message" => "Invalid response from server" }
       end
 
-      { message: "Booking availability sent successfully" }
+      {
+        statusCode: response.code.to_i,
+        data: data
+      }
+    rescue => e
+      e.message
     end
 
-    # Bulk bookings
     def send_multiple_bookings(bookings)
-      raise ArgumentError, "Bookings array is required" unless bookings.is_a?(Array) && !bookings.empty?
+      return "Bookings array is required" unless bookings.is_a?(Array) && !bookings.empty?
 
-      valid_statuses = ["available", "unavailable"]
       failed_bookings = []
+      successful_bookings = []
 
       bookings.each do |booking|
-        date = booking[:date]
-        service_type = booking[:serviceType]
-        status = booking[:status]
-
-        # Validate fields
-        if date.nil? || service_type.nil? || status.nil?
-          failed_bookings << booking.merge(error: "Missing required fields")
-          next
-        end
-
-        unless valid_date?(date)
-          failed_bookings << booking.merge(error: "Invalid date format, expected YYYY-MM-DD")
-          next
-        end
-
-        unless valid_statuses.include?(status)
-          failed_bookings << booking.merge(error: "Invalid status value")
+        result = send_booking(
+          date: booking[:date],
+          service_type: booking[:serviceType],
+          status: booking[:status]
+        )
+
+        # local validation / network errors
+        if result.is_a?(String)
+          failed_bookings << booking.merge(error: result)
           next
         end
 
-        begin
-          send_booking(date: date, service_type: service_type, status: status)
-        rescue => e
-          failed_bookings << booking.merge(error: e.message)
+        # API error response
+        if !result[:statusCode] || result[:statusCode] >= 400
+          failed_bookings << booking.merge(error: result)
+        else
+          successful_bookings << result
         end
       end
 
-      if failed_bookings.empty?
-        { message: "Booking availability sent successfully" }
-      else
-        { message: "Some bookings failed", failedBookings: failed_bookings }
-      end
+      {
+        message: failed_bookings.empty? ?
+          "All bookings sent successfully" :
+          "Some bookings failed",
+        successfulBookings: successful_bookings,
+        failedBookings: failed_bookings
+      }
     end
   end
 end

data/lib/servizehub/version.rb

@@ -1,3 +1,3 @@
 module Servizehub
-  VERSION = "1.0.1"
+  VERSION = "1.0.2"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: servizehub
 version: !ruby/object:Gem::Version
-  version: 1.0.1
+  version: 1.0.2
 platform: ruby
 authors:
 - Hariraghav S