Vendor specific fields #275
-
From the meeting on October 4, 2022:
Previous Agreements
Discussion Points
Potential Solutions if we decide to do vendor properties
|
Beta Was this translation helpful? Give feedback.
Replies: 5 comments 4 replies
-
If i missed anything or said something incorrectly, please correct me! |
Beta Was this translation helpful? Give feedback.
-
Thanks for recording these notes @naveedn. I'm still not convinced we want or need this capability, because I feel it encourages us to continue operating in a world of everybody supporting N data models instead of 1 data model. I do see the point about adoption, but the tension @robwise mentioned is strong -- once people come to rely on vendor-specific properties, pretty much nobody is going to want to stop and do maintenance to pull those into the standard and deprecate them. IF we are to allow embed vendor properties on objects, I like your last option best (support a top-level vendor-properties object on each OCF Object). I would suggest two small tweaks to your example: Add a "wart" to the key as a visual reminder that it's not a part of the schema proper, and use domain names for vendor keys. e.g.: "_vendor": {
"carta.com": { },
"ltse.com": { }
} I have another suggestion that I think is less harmful to the schema, and that is to add a new File to the package. This file would contain only vendor specific data, and would use object references to do so. For example, let's say Example, Inc wants to include more data about the VALUATION object in https://github.com/Open-Cap-Table-Coalition/Open-Cap-Format-OCF/blob/616d972f4ce99f33ba2729203e56ea5c0acb2017/samples/Valuations.ocf.json. They could do so by emitting an {
"file_type": "OCF_VENDOR_FILE",
"vendor": "example.com",
"items": [
{
"id": "892e8b19-ed26-4049-9ada-a5cf816376fd",
"object_type": "VALUATION"
"phase_of_moon": "WAXING_GIBBOUS",
"the_answer": 42
}
]
} The benefits here are:
Thoughts? |
Beta Was this translation helpful? Give feedback.
-
I personally like the last option |
Beta Was this translation helpful? Give feedback.
-
What about something like a
I think this is similar to one of the options @naveedn proposed above (thanks for the great write-up, @naveedn) |
Beta Was this translation helpful? Give feedback.
-
Per TWG Discussion 10/18/22:
|
Beta Was this translation helpful? Give feedback.
Per TWG Discussion 10/18/22: