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

TradeUpdate event id is null #151

Open
drse opened this issue Mar 26, 2024 · 3 comments
Open

TradeUpdate event id is null #151

drse opened this issue Mar 26, 2024 · 3 comments
Assignees

Comments

@drse
Copy link

drse commented Mar 26, 2024

TradeUpdate appears to map the event id to the execution_id field, whereas the api is returning an id field. Seeing this when using updatesStream():

alpacaAPI.updatesStream().setListener(tradeUpdate -> {
    assert tradeUpdate.getData().getExecutionId() != null;
});
@Petersoj
Copy link
Owner

Relevant code here. When I was testing this, I was receiving execution_id, but perhaps Alpaca has recently changed it to id. I'll look into this in the near future.

@Petersoj Petersoj self-assigned this Apr 14, 2024
@Petersoj
Copy link
Owner

Petersoj commented Aug 3, 2024

@drse Is this still an issue for you?

@drse
Copy link
Author

drse commented Aug 8, 2024

Yes, I'm currently using a workaround to hash the event contents in place of the id.

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

No branches or pull requests

2 participants