-
Notifications
You must be signed in to change notification settings - Fork 11
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
Fix proposal's websocket specs #279
Conversation
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.
LGTM. lots of good catches and updates, thanks!!!
| [request/read_cache](#read-cache) | [response/reading](#reading) | | ||
| [request/write_async](#write-asynchronous) | [response/write_async](#write-asynchronous-1) | | ||
| [request/write_sync](#write-synchronous) | [response/write_sync](#write-synchronous-1) | | ||
| [request/transaction](#transaction) | [response/transaction](#transaction-1) | |
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.
I was JUST thinking how it would be useful to have a TOC (:
| :---- | :--- | :------- | :---------- | | ||
| *start* | `string` | no | An RFC3339 formatted timestamp which specifies a starting bound on the cache data to return. | | ||
| *end* | `string` | no | An RFC3339 formatted timestamp which specifies an ending bound on the cache data to return. | | ||
|
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.
whoops. this is a big omission on my part. good catch.
* Fix returned data scheme in response/device_summary * Fix returned data scheme in response/read * Add toc for websocket events * Fix table styling 1 * Fill in readcache event data * Add version event * Add note for plugin event * Add read device event * Update write sync event * Add write async event * Update transaction event
* Fix returned data scheme in response/device_summary * Fix returned data scheme in response/read * Add toc for websocket events * Fix table styling 1 * Fill in readcache event data * Add version event * Add note for plugin event * Add read device event * Update write sync event * Add write async event * Update transaction event
This PR fix websocket specs that fall out of this PR discussion.