-
Notifications
You must be signed in to change notification settings - Fork 6
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
Readme update #115
Readme update #115
Conversation
…ature/examples # Conflicts: # package.json # yarn.lock
…ature/examples
…ature/examples
8353510
to
6ef605c
Compare
6ef605c
to
3814b03
Compare
…-sdk into feature/readme-update
…ature/readme-update # Conflicts: # examples/cra/src/App.tsx # examples/nodejs/src/index.ts # examples/vanilla/src/index.ts
ESLint Summary View Full Report
Report generated by eslint-plus-action |
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 see now i only half reviewed this and left unsent comments.
anyways, most of my comments are now in #117
It looks great!
- The CoW API (`cowSdk.cowApi`) | ||
- The CoW Subgraph (`cowSdk.cowSubgraphApi`) | ||
- Convenient method to facilitate signing orders (i.e `cowSdk.signOrder`) | ||
- `OrderBookApi` - provides the ability to retrieve orders and trades from the CowSap order-book, as well as add and cancel them |
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.
nit: i think order-book doesn't have highphen
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.
Also you can post signed orders :)
Updated the Readme corresponding to the dogfooding changes.
Added quick start examples.
New content