search

jugaad-py / jugaad-trader

Unofficial python client for Zerodha
https://marketsetup.in/documentation/jugaad-trader/
157 stars 112 forks source link

Updated payload to fix the web-socket issues #22

Closed iamknownstranger closed 3 years ago

sevakram commented 3 years ago

@iamknownstranger , I saw the changes, I am yet to try it. Please let me know if it worked, I'll merge the PR

iamknownstranger commented 3 years ago

Hello, I'm a bit confused while setting up the local dev environment. Currently, I had to uninstall and reinstall the package using setup.py every time while debugging. I truly appreciate it if you can include the dev environment setup which is quite helpful for anyone else who is interested to contribute.

sevakram commented 3 years ago

@iamknownstranger , I have written a small document of my development workflow (https://github.com/jugaad-py/jugaad-trader/blob/master/contributing.md).

You should not have to install every time you make change. I usually write tests to validate if the changes are correct (and run pytest) or create a file temp.py in the root folder (alongside setup.py). With this `import jugaad_trader' should work normally.

And to run cli jtrader, you have to run python -m jugaad_trader.cli

Hope this helps.

Sorry for delayed responses.