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

해외주식 실시간 체결 이벤트 버그 수정 #53

Merged
merged 3 commits into from
Nov 6, 2024

Conversation

Soju06
Copy link
Owner

@Soju06 Soju06 commented Nov 6, 2024

🛠️ PR Summary

🌟 요약

어떤 것이 변경되었나요? 간략히 설명해주세요.

정상적으로 해외주식 실시간 체결 이벤트를 수신할 수 있습니다.

📊 주요 변경 사항

주요 변경 사항을 적어주세요.

  • pykis/api/account/order.pypykis/event/filters/order.py에서 해외 주문번호의 지점코드(원주문번호)를 비교하지 않도록 변경하였습니다.
  • pykis/responses/websocket.pyKisWebsocketResponse 파싱 로직에 KisType.default 값을 활용하도록 변경하였습니다.
  • pykis/api/websocket/order_execution.pyKisForeignRealtimeOrderExecution에서 접수 주문시 ODER_QTY가 없을 때 발생하는 파싱 오류를 해결했습니다.

🎯 목적 및 영향

  • 목적: 왜 이 PR이 필요한가요?
    정상적으로 해외주식 실시간 체결 이벤트를 수신할 수 있습니다.

  • 영향: 이 변경 사항이 어떤 영향을 미치나요?
    정상적으로 해외주식 실시간 체결 응답을 파싱하고, 필터링 및 이벤트를 수신할 수 있습니다.

@Soju06 Soju06 added the 버그 뭔가 작동하지 않습니다. label Nov 6, 2024
@Soju06 Soju06 self-assigned this Nov 6, 2024
@Soju06 Soju06 linked an issue Nov 6, 2024 that may be closed by this pull request
2 tasks
@Soju06 Soju06 merged commit f8380f2 into main Nov 6, 2024
1 check passed
@Soju06 Soju06 deleted the fix/44-foreign-realtime-order-execution-event branch November 7, 2024 02:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
버그 뭔가 작동하지 않습니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[버그]: 해외주식 실시간 체결통보
1 participant