dymaptic / GeoBlazor

Core Components for the Dymaptic GeoBlazor Library
MIT License
108 stars 19 forks source link

AttributesDictionary Json Converter is not reversible #350

Closed TimPurdum closed 1 month ago

TimPurdum commented 1 month ago

The AttributesDictionaryConverter was written primarily to support serialization to JavaScript. Because of that, the Write output format is not a normal Dictionary/Object, and it is not supported by the Read method.

The custom Write logic probably really belongs in the TypeScript layer. However, it stores .NET type information, and might need to stay where it is (DateTimes, for example, once serialized, look like strings, and we lose the strong typing). Either way, the Read method should support both dictionary/object and array formats, so that it can accept data from different sources.

TimPurdum commented 1 month ago

Closed with #349