Fix grpc rule logic

Author: Knerio

app/services/runtimes/data_types/update_service.rb

@@ -159,18 +159,16 @@ def update_rules(rules, data_type)
         db_rules = data_type.rules.first(rules.length)
         rules.each_with_index do |rule, index|
           config = rule.rule_config.to_h.transform_values do |value|
-            case value
-            when Tucana::Shared::DataTypeIdentifier
-              if value.data_type_identifier.present?
-                value.data_type_identifier
-              elsif value.generic_key.present?
-                { generic_key: value.generic_key }
-              else
-                { generic_type: value.generic_type }
-              end
-            else
-              value
-            end
+            next value unless value.is_a?(Hash)
+
+            data_type_identifier = value[:data_type_identifier]
+            generic_key = value[:generic_key]
+
+            next value if data_type_identifier.nil? || generic_key.nil? # return the old thing if neither is present
+            next data_type_identifier unless data_type_identifier.empty? # return data_type_identifier if it's present
+            next { generic_key: generic_key } if generic_key.present? # return generic_key if it's present
+
+            next { generic_type: value[:generic_type] }
           end
 
           db_rules[index] ||= DataTypeRule.new

app/services/runtimes/flow_types/update_service.rb

@@ -36,6 +36,8 @@ def execute
 
           UpdateRuntimeCompatibilityJob.perform_later({ runtime_id: current_runtime.id })
 
+          logger.info(message: 'Updated flow types for runtime', runtime_id: current_runtime.id)
+
           ServiceResponse.success(message: 'Updated data types', payload: flow_types)
         end
       end