-
Notifications
You must be signed in to change notification settings - Fork 2
Feat/realitybot #128
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
Feat/realitybot #128
Conversation
issue: #91
- 업비트 호가 정책에 따른 단가 적용 issue : #91
- fixed-rate : 5초마다 주문 실행 - order-level : 한번 실행 시 숫자 갯수만큼 실행 및 오더북에 해당하는 주문 - corn : 단가 수집 기간, 주문 가격 단위 설정 issue : #93
…tybot # Conflicts: # src/main/java/com/cleanengine/coin/realitybot/service/OrderGenerateService.java
- 매일 자정으로 재수정
- 초기 선형 보간 및 refactor
- 코인원 추가 issue: #39
…nto feat/realitybot
caniro
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
테스트랑 리팩토링 많이 하셨네요.. 고생하셨습니다!
src/main/java/com/cleanengine/coin/realitybot/api/BithumbAPIClient.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이 클래스는 테스트에서만 사용되는 건가요??
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
넵 시세를 이탈 시켰을 때 어떻게 작동하는 지 보는 테스트 용도가 맞습니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
개인적으로 프로덕션 코드랑 테스트 코드는 분리되는 게 나을 것 같은 느낌이 드네요
src/main/java/com/cleanengine/coin/realitybot/api/UnitPriceRefresher.java
Outdated
Show resolved
Hide resolved
caniro
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
빌드 에러 수정까지 고생하셨습니다~

✨ 작업내용
🐞 이슈사항