-
Notifications
You must be signed in to change notification settings - Fork 126
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
Format improvements #415
Format improvements #415
Conversation
|
||
Additionally, using these endpoints requires your global company ID in each request. You can find your global company ID by using the [Discovery API](../discovery.md). | ||
Additionally, using these endpoints requires your global company ID in each request. You can find your global company ID by using the [Discovery API](../discovery.md) |
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.
Period is actually needed here.
@@ -454,7 +454,7 @@ The following table describes the GET classification template file request param | |||
| Name | Required | Type | Description | | |||
| --- | --- | --- | --- | | |||
| `dataset_id` | required | string | Classification dataset ID | | |||
| `format` | optional | string | Template format. The default is `tsv`. Includes the enums: `tsv`, and `csv`. | | |||
| `format` | optional | string | Template format. The default is `tsv`. Possible values include `tsv` or `csv`. | |
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.
Will need articulation of data type enums.
"state": "completed", | ||
"recipients": [ | ||
"[email protected]" | ||
] |
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.
The orphan single quote indeed suggests a JSON error. This is a direct value taken from Tony so I will double-check with him after merge to verify.
} | ||
} | ||
] | ||
}' |
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.
See note on orphan single quote in other topic.
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.
Inline comments are for fixes post merge.
Description
Checklist