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

[BUGFIX] order key error. #56

Merged
merged 1 commit into from
Jan 14, 2022

Conversation

resoliwan
Copy link
Contributor

가끔 이벤트 수신 순서가 바뀌는 경우가 존재.
키움 명세서에도 이벤트 순서가 바뀔 수 있다고 명세 되어 있습니다.
처리 완료 chenjan 데이터를 받을 떄까지 최대 5번 확인합니다.
그 안에 키가 있다면 정상처리 그렇지 않다면 이전과 같이 에러를 발생시킵니다.

가끔 이벤트 수신 순서가 바뀌는 경우가 존재.
키움 명세서에도 이벤트 순서가 바뀔 수 있다고 명세 되어 있습니다.
처리 완료 chenjan 데이터를 받을 떄까지 최대 5번 확인합니다.
그 안에 키가 있다면 정상처리 그렇지 않다면 이전과 같이 에러를
발생시킵니다.
@codecov
Copy link

codecov bot commented Jan 3, 2022

Codecov Report

Merging #56 (d604ae9) into master (1097910) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #56      +/-   ##
=========================================
- Coverage    4.32%   4.32%   -0.01%     
=========================================
  Files         156     156              
  Lines       11221   11223       +2     
=========================================
  Hits          485     485              
- Misses      10736   10738       +2     
Flag Coverage Δ
windows-latest-py3.7 4.05% <0.00%> (-0.01%) ⬇️
windows-latest-py3.8 4.32% <0.00%> (-0.01%) ⬇️
windows-latest-py3.9 4.32% <0.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
koapy/backtrader/KiwoomOpenApiPlusStore.py 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1097910...d604ae9. Read the comment docs.

@elbakramer elbakramer merged commit 5a9d6a9 into elbakramer:master Jan 14, 2022
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.

2 participants