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.

Sign up for GitHub

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

Jump to bottom

Real-time News Data Stream via WebSocket Subscription #392

Open
wants to merge 33 commits into
base: master
Choose a base branch
from
Open

Real-time News Data Stream via WebSocket Subscription #392

wants to merge 33 commits into from
+378 −81

Conversation

IsaacTrevino
Copy link

Reopened this Feature in separate PR, due to renaming issues.

IsaacTrevino and others added 25 commits October 19, 2023 22:48
@IsaacTrevino
Added News Stream
5cbc8d6
@IsaacTrevino
updated readme
2a66fd6
@IsaacTrevino
Remove NEWS_DATA_STREAM
35b298f
@IsaacTrevino
poetry pre-commit
1245a57
@IsaacTrevino
update readme
78fa279
@IsaacTrevino
test_cast
75eba6a
@IsaacTrevino
test_cast
bef4760
@IsaacTrevino
test_cast
48d50f2
@IsaacTrevino
test_cast
710ef11
@IsaacTrevino
test_cast
c06a16c
@IsaacTrevino
test_cast
b9bde00
@IsaacTrevino
Merge pull request #1 from IsaacTrevino:test_cast
5308187 
Added News Test Case
@IsaacTrevino
clean up test_case.py imports
b4feec9
@IsaacTrevino
Merge branch 'alpacahq:master' into master
2dc6313
@IsaacTrevino
fixed news model issues
8a8456d
@IsaacTrevino
Merge branch 'master' of https://github.com/IsaacTrevino/alpaca-py
8aba3d9
@IsaacTrevino
news websocket.py _dispatch fix
a0f3dad
@IsaacTrevino
websocket raw_data=false testing
3d0918c
@IsaacTrevino
fixed timestamp to datetime in news ws
6b0cb4b
@IsaacTrevino
remove any
c7c4286
@IsaacTrevino
revert NewsClient namechange
a3e9396
@IsaacTrevino
Added TimeSeriesMixin to News model
16eb391
@IsaacTrevino
set news.images default none
4924843
@IsaacTrevino @hiohiohio
Update README.md
1c81fc9 
Co-authored-by: hiohiohio <hiohiohio@users.noreply.github.com>
@IsaacTrevino
Merge branch 'alpacahq:master' into news-stream
aacc3f3
hiohiohio
hiohiohio reviewed Dec 26, 2023
View reviewed changes
Copy link
Contributor

@hiohiohio hiohiohio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sorry for late reviewing. left some comments.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
alpaca/common/websocket.py Outdated Show resolved Hide resolved
alpaca/data/enums.py Outdated Show resolved Hide resolved
alpaca/data/historical/news.py Outdated Show resolved Hide resolved
alpaca/data/historical/news.py Outdated Show resolved Hide resolved
alpaca/data/live/news.py Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hiohiohio hiohiohio Awaiting requested review from hiohiohio hiohiohio is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@IsaacTrevino @hiohiohio