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

Resolve "Add processBefore parameter to kraken.futures.Trade.{cancel_order,edit_order,create_order,create_batch_order}" #192

Conversation

btschwertfeger
Copy link
Owner

Added the new parameters.

This PR must wait before merge until #189 is merged, since the still broken futures tests will fail and block the merge.

@btschwertfeger btschwertfeger added enhancement New feature or request Futures Topic related to Futures trading labels Feb 21, 2024
@btschwertfeger btschwertfeger added this to the Future Release milestone Feb 21, 2024
@btschwertfeger btschwertfeger self-assigned this Feb 21, 2024
@btschwertfeger btschwertfeger marked this pull request as draft March 1, 2024 17:48
@btschwertfeger btschwertfeger marked this pull request as ready for review March 8, 2024 20:44
…uturestradecancel_orderedit_ordercreate_ordercreate_batch_order
Copy link

codecov bot commented Mar 9, 2024

Codecov Report

Attention: Patch coverage is 88.88889% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 89.45%. Comparing base (3f53ec7) to head (e3ed0d7).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #192      +/-   ##
==========================================
- Coverage   89.46%   89.45%   -0.01%     
==========================================
  Files          21       21              
  Lines        1879     1888       +9     
==========================================
+ Hits         1681     1689       +8     
- Misses        198      199       +1     
Flag Coverage Δ
unittests 89.45% <88.88%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
kraken/futures/trade.py 94.31% <88.88%> (-0.62%) ⬇️

@btschwertfeger btschwertfeger merged commit 0419f08 into master Mar 9, 2024
32 checks passed
@btschwertfeger btschwertfeger deleted the 191-add-processbefore-parameter-to-krakenfuturestradecancel_orderedit_ordercreate_ordercreate_batch_order branch March 9, 2024 06:00
btschwertfeger added a commit that referenced this pull request Mar 12, 2024
…ncel_order`,`edit_order`,`create_order`,`create_batch_order`}" (#192)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Futures Topic related to Futures trading
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add processBefore parameter to kraken.futures.Trade.{cancel_order,edit_order,create_order,create_batch_order}
1 participant