fixes

lib/kennel/importer.rb

@@ -2,8 +2,15 @@
 
 module Kennel
   class Importer
+    # title will have the lock symbol we need to remove when re-importing
     TITLES = [:name, :title].freeze
-    SORT_ORDER = [*TITLES, :id, :kennel_id, :type, :tags, :query, :sli_specification, *Models::Record.subclasses.map { |k| k::TRACKING_FIELDS }, :template_variables].freeze
+
+    # bring important fields to the top
+    SORT_ORDER = [
+      *TITLES, :id, :kennel_id, :type, :tags, :query, :sli_specification,
+      *Models::Record.subclasses.flat_map { |k| k::TRACKING_FIELDS },
+      :template_variables
+    ].freeze
 
     def initialize(api)
       @api = api
@@ -19,15 +26,11 @@ def import(resource, id)
         raise(ArgumentError, "#{resource} is not supported")
 
       data = @api.show(model.api_resource, id)
-      # get sli specification before normalization.
-      sli_specification = data[:sli_specification] || data.dig(:sli_specification, :time_slice)
 
       id = data.fetch(:id) # keep native value
       model.normalize({}, data) # removes id
       data[:id] = id
 
-      data[:sli_specification] = sli_specification if sli_specification
-
       title_field = TITLES.detect { |f| data[f] }
       title = data.fetch(title_field)
       title.tr!(Kennel::Models::Record::LOCK, "") # avoid double lock icon

lib/kennel/models/slo.rb

@@ -4,7 +4,10 @@ module Models
     class Slo < Record
       include TagsValidation
 
-      READONLY_ATTRIBUTES = superclass::READONLY_ATTRIBUTES + [:type_id, :monitor_tags, :target_threshold, :timeframe, :warning_threshold]
+      READONLY_ATTRIBUTES = [
+        *superclass::READONLY_ATTRIBUTES,
+        :type_id, :monitor_tags, :target_threshold, :timeframe, :warning_threshold
+      ].freeze
       TRACKING_FIELD = :description
       DEFAULTS = {
         description: nil,
@@ -35,8 +38,8 @@ def build_json
           type: type
         )
 
-        if type == 'time_slice'
-          data[:sliSpecification] = :sli_specification
+        if type == "time_slice"
+          data[:sli_specification] = sli_specification
         elsif v = query
           data[:query] = v
         end