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

kraken.spot.Market.get_recent_trades: 'since' parameter does not work #119

Closed
andyDoucette opened this issue Jun 28, 2023 · 4 comments · Fixed by #120
Closed

kraken.spot.Market.get_recent_trades: 'since' parameter does not work #119

andyDoucette opened this issue Jun 28, 2023 · 4 comments · Fixed by #120
Assignees
Labels
Bug Something isn't working
Milestone

Comments

@andyDoucette
Copy link

andyDoucette commented Jun 28, 2023

I'm using version 1.4.0

https://docs.kraken.com/rest/#tag/Market-Data/operation/getRecentTrades says that the since parameter "Filter trades since given timestamp".

Supplying the since parameter has no effect.

I can see in the code that:

362             params: dict = {"pair": pair}
363             if defined(since):
364                 params["since"] = None

I wonder if line 364 should be params["since"] = since instead, or something like that? Seems like it's being set to None no matter what, which is probably why the feature isn't working.

@andyDoucette andyDoucette added the Bug Something isn't working label Jun 28, 2023
@btschwertfeger
Copy link
Owner

kraken.spot.User.get_trades_history do not use the the since parameter (kraken/spot/user.py#L468-L538). This is also not part of the documentation you linked.

start is what you are looking for.

@andyDoucette
Copy link
Author

Hey sorry. I sent the wrong docs link. The title and rest of the description was intended.

@btschwertfeger
Copy link
Owner

I see - and will fix that. Thank you!

@andyDoucette
Copy link
Author

andyDoucette commented Jun 28, 2023 via email

@btschwertfeger btschwertfeger self-assigned this Jun 28, 2023
@btschwertfeger btschwertfeger changed the title Market().get_recent_trades() since parameter does not work. kraken.spot.Market.get_recent_trades: 'since' parameter does not work Jun 28, 2023
@btschwertfeger btschwertfeger added this to the v1.4.1 milestone Jun 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants