mirror of
https://github.com/d0zingcat/gocryptotrader.git
synced 2026-05-14 07:26:47 +00:00
1212b6750e
* OrderManager: Fix race condition in submit with ws If the ws sees the order before processSubmittedOrder then it will have assigned it an internal order id already and added it to the store. Don't treat that as an error. Instead just use the newer ws details * OrderManager: Fix error comparisson Should always use errors.Is when possible * Tests: Simplify btcusd test pair declaration * OrderManager: Improve test readability * OrderManager: Add orderstore.getByDetail test * Return a fresh pointer from orderstore.getByDetail This protects the order.Details in the store from direct access. The use-case was to allow the returned objects to be references so that future changes to them would be reflected. However we're not ready yet to allow people to touch the orders directly, because they're not protected directly by a mutex, and nothing would stop consumers contaminating the integrity of the data. We can revisit this topic later atomicly, but it's definitely tangental to the cause of action for PR #1336. * Fix GetByDetail tests to assert a new pointer * OrderManager: Avoid possible lock races This fix internalises the getByDetail because the implication of moving lock ownership out of exists/getByDetail to consumers breaks the order store struct encapsulation in a way we really don't want to. It's also more efficient * Fix spelling mistake Co-authored-by: Scott <gloriousCode@users.noreply.github.com> * OrderManager: Fix TestSubmitOrder... description * OrderManager: Improve clarity of comment * OrderManager: Capitalise error message On failure to add to orderstore, capitalise the error message Co-authored-by: Adrian Gallagher <adrian.gallagher@thrasher.io> --------- Co-authored-by: Scott <gloriousCode@users.noreply.github.com> Co-authored-by: Adrian Gallagher <adrian.gallagher@thrasher.io>
GoCryptoTrader package Exchanges
This exchanges package is part of the GoCryptoTrader codebase.
This is still in active development
You can track ideas, planned features and what's in progress on this Trello board: https://trello.com/b/ZAhMhpOy/gocryptotrader.
Join our slack to discuss all things related to GoCryptoTrader! GoCryptoTrader Slack
Current Features for exchanges
-
This package is used to connect and query data from supported exchanges.
-
Please checkout individual exchange README for more information on implementation
Guide for adding a new exchange
- A guide on implementing API support for a new exchange can be found here
websocket notes
- If contributing websocket improvements, please make sure order reports follow these rules.
Please click GoDocs chevron above to view current GoDoc information for this package
Contribution
Please feel free to submit any pull requests or suggest any desired features to be added.
When submitting a PR, please abide by our coding guidelines:
- Code must adhere to the official Go formatting guidelines (i.e. uses gofmt).
- Code must be documented adhering to the official Go commentary guidelines.
- Code must adhere to our coding style.
- Pull requests need to be based on and opened against the
masterbranch.
Donations
If this framework helped you in any way, or you would like to support the developers working on it, please donate Bitcoin to:
bc1qk0jareu4jytc0cfrhr5wgshsq8282awpavfahc