Skip to content
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

Added 'public client' & enhanced orders #3

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

blainecostello
Copy link
Contributor

At the end of client.py I've added some function definitions for more user-friendly order functionality to handle all the bs we've been dealing with (timestamps, storageID, etc.)

I've also added a bunch of global functions that serve as a 'public client' where users can go to get public information without impacting their rate limit on the authenticated client. These are just a bunch of simple requests packaged as easy-to-use functions that return simple json data. More documentation needs to be added, as it will be copied from the REST api doc provided by loopring.

blainecostello and others added 2 commits January 20, 2022 01:14
At the end of client.py I've added some function definitions for more user-friendly order functionality to handle all the bs we've been dealing with (timestamps, storageID, etc.)

I've also added a bunch of global functions that serve as a 'public client' where users can go to get public information without impacting their rate limit on the authenticated client.  These are just a bunch of simple requests packaged as easy-to-use functions that return simple json data.  More documentation needs to be added, as it will be copied from the REST api doc provided by loopring.
@DiggidyDev
Copy link
Owner

DiggidyDev commented Jan 20, 2022

It could be worth using aiohttp instead of requests for the global functions you made, to prevent any blocking calls from being made (although that would also partially be down to user error). There is however a plan to also make a synchronous branch of the wrapper, so they'll be perfect to use in there!

There's also the option to allow sync requests to execute concurrently by using asyncio's loop.run_in_executor().

Great functions though!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants