Prototype of JsonModel<T> abstract class#43571
Prototype of JsonModel<T> abstract class#43571annelo-msft wants to merge 5 commits intoAzure:mainfrom
Conversation
| } | ||
| public partial interface IJsonModel | ||
| { | ||
| System.Collections.Generic.IDictionary<string, object> AdditionalProperties { get; } |
There was a problem hiding this comment.
would it be appropriate that we call it AdditionalProperties?
A generated model in our current generated libraries might have AdditionalProperties property if this is defined in the spec, and its type is usually IDictionary<string, BinaryData> or IReadOnlyDictionary<string, BinaryData>
There was a problem hiding this comment.
That is a good point, and this is currently an open question under discussion - whether we should merge this feature with the existing AdditionalProperties feature. It is currently unclear whether we would want multiple property dictionaries on a type -- it sounds like we would prefer a single dictionary that supported both/all three cases (with _serializedAdditionalRawData as well), but the details of how this would work have not been finalized.
|
Hi @annelo-msft. Thank you for your interest in helping to improve the Azure SDK experience and for your contribution. We've noticed that there hasn't been recent engagement on this pull request. If this is still an active work stream, please let us know by pushing some changes or leaving a comment. Otherwise, we'll close this out in 7 days. |
github-actions
Bot
added
the
no-recent-activity
|
Hi @annelo-msft. Thank you for your contribution. Since there hasn't been recent engagement, we're going to close this out. Feel free to respond with a comment containing |
2 participants
No description provided.