Skip to content

Commit

Permalink
cleanup
Browse files Browse the repository at this point in the history
  • Loading branch information
Ondřej Chrastina committed Mar 6, 2023
1 parent 4c8ab5e commit ba5ee9a
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions Kontent.Ai.Delivery/ContentItems/ModelProvider.cs
Original file line number Diff line number Diff line change
Expand Up @@ -230,7 +230,7 @@ private ContentItemElements GetAllPropertiesValuesAsync(JObject elementsData, Pr
var element = item.Value;

// TODO think about value converter implementation
// TODO what about codename property - now it is null
// TODO what about codename property - now it is not null, bit withCodename is not really nice
IContentElementValue value = (element["type"].ToString()) switch
{
// TODO do we want to use string/structured data for rich text - probably think about support both ways
Expand All @@ -248,9 +248,6 @@ private ContentItemElements GetAllPropertiesValuesAsync(JObject elementsData, Pr
"text" => element.ToObject<TextElementValue>(Serializer).WithCodename(key),
_ => throw new ArgumentException($"Argument type ({element["type"].ToString()}) not supported.")
};

// TODO Fix the empty Codename? Probably yes (wrap ToObject with normalization logic)
// value.Codename = key;
result.Add(key, value);
}
return result;
Expand Down

0 comments on commit ba5ee9a

Please sign in to comment.