-
Notifications
You must be signed in to change notification settings - Fork 39
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs: documenting the different types of Object Definitions #3135
Conversation
* A CSV of a String | ||
* A List of a String |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we provide an actual example of these? like is the CSV apple,pear,strawberry
but a list of a string is ["apple","pear","strawberry"]
?
|
||
Notes: | ||
|
||
* At present the Object Definition type is reused, but shouldn't be, as we'll want to extend this in the future when we parse Data Plane (which has way more possible values for types). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we be explicit about what specifically is missing here? e.g.
- add an Options Object Definition type
- add a Terraform Object Definition type
|
||
Notes: | ||
|
||
* In retrospect this should probably be named `SDKObjectDefinition` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this a rename that we should undertake? It's unclear what this sentence is trying to convey or add.
In the end we ended up deciding to document these inline within the new SDK types, so I'm going to close this out:
|
No description provided.