Skip to content

Fix Ticker.news example in v3 migration guide - #233

Open
mahimn01 wants to merge 1 commit into
ib-api-reloaded:next-protobuffrom
mahimn01:docs/v3-ticker-news-example
Open

mahimn01 wants to merge 1 commit into
ib-api-reloaded:next-protobuffrom
mahimn01:docs/v3-ticker-news-example

Conversation

@mahimn01

Copy link
Copy Markdown

Hi Matt,

The Ticker.news example in docs/migration/v3.md prints tick.time, but NewsTick has timeStamp and no time field, so the snippet raises AttributeError. This changes it to tick.timeStamp and drops the unused deque import.

Thanks!

This branch has not been deployed

No deployments
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.

1 participant