(For 2973) Remove Children/Namedchildren#45
Conversation
Kasdejong
left a comment
There was a problem hiding this comment.
Does this mean no one from the outside will be able to access the IReadOnlyDictionary fields anymore? do we need them in the SDK then? I see they are all internal now
For now, that is correct. We will have a public interface that we are designing right now that will act as "the" Base interface to navigate to children. But I intend to not have IReadOnlyDictionary return as an interface on the POCO's. It's too noisy, and some frameworks (and debuggers) are tempted to "smart" things with it (like display it as a dictionary in the debugger) that I really don't want to. But let's see what kind of feedback we get when we remove IDictionary ;-) |
Code generation changes for FirelyTeam/firely-net-sdk#2973