Allow passing of nil for external_id

External IDs can actually be set to null. On create this isn't an issue,
not sending it will result in it being null. But on update if a user
passes nil we want to be sure a null value goes to the API.

Since organizations and user update methods are different I opted for a
slightly difference solution in each place.

Author: ericroberts

lib/workos/organizations.rb

@@ -139,15 +139,13 @@ def update_organization(
         domain_data: nil,
         domains: nil,
         name: nil,
-        external_id: nil,
+        external_id: :not_set,
         allow_profiles_outside_organization: nil
       )
-        body = {
-          name: name,
-          external_id: external_id,
-        }
+        body = { name: name }
         body[:domain_data] = domain_data if domain_data
         body[:stripe_customer_id] = stripe_customer_id if stripe_customer_id
+        body[:external_id] = external_id if external_id != :not_set
 
         if domains
           warn_deprecation '`domains` is deprecated. Use `domain_data` instead.'

lib/workos/user_management.rb

@@ -234,14 +234,14 @@ def create_user(
       # @return [WorkOS::User]
       def update_user(
         id:,
-        email: nil,
-        first_name: nil,
-        last_name: nil,
-        email_verified: nil,
-        external_id: nil,
-        password: nil,
-        password_hash: nil,
-        password_hash_type: nil
+        email: :not_set,
+        first_name: :not_set,
+        last_name: :not_set,
+        email_verified: :not_set,
+        external_id: :not_set,
+        password: :not_set,
+        password_hash: :not_set,
+        password_hash_type: :not_set
       )
         request = put_request(
           path: "/user_management/users/#{id}",
@@ -254,7 +254,7 @@ def update_user(
             password: password,
             password_hash: password_hash,
             password_hash_type: password_hash_type,
-          }.compact,
+          }.reject { |_, v| v == :not_set },
           auth: true,
         )